Skip to content

Web Accessibility#11

Open
DerekJB1987 wants to merge 1 commit intomasterfrom
Web_Accessibility
Open

Web Accessibility#11
DerekJB1987 wants to merge 1 commit intomasterfrom
Web_Accessibility

Conversation

@DerekJB1987
Copy link
Collaborator

@DerekJB1987 DerekJB1987 commented Feb 14, 2019

Issue #1 Web_Accessibility

To create web accessibility functionality I added a href link in the footer
to take admin and registered users to the WebAIM Accessibility article
detailing in AltText since that was outlined in our acceptance criteria.

Additional Information

  • None required to work with anyone's local dev environment.

Requested Guidance

  • Please review the href link in the footer and verify that it directs the person to the WebAIM website and specifically the article on accessibility and AlternateText.

@DerekJB1987 DerekJB1987 changed the title Added href link for an accessibility guide accessibility Feb 14, 2019
@DerekJB1987 DerekJB1987 changed the title accessibility Web_Accessibility Feb 14, 2019
@DerekJB1987 DerekJB1987 changed the title Web_Accessibility Web Accessibility Feb 17, 2019
<footer>
<p>&copy; Company 2015</p>

<a href="https://webaim.org/techniques/alttext/">WebAIM Accessibility Guide</a>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest adding the id tag of "basics" to the end of this URL to send the User directly to the "Alternative Text Basics" section.

Suggested change
<a href="https://webaim.org/techniques/alttext/">WebAIM Accessibility Guide</a>
<a href="https://webaim.org/techniques/alttext/#basics">WebAIM Accessibility Guide</a>

@dineenlauren
Copy link
Collaborator

No issues with code. Ready to be merged into Master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants