Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Linked Image (atum) a11y #34768

Merged
merged 1 commit into from
Jul 21, 2021
Merged

[4.0] Linked Image (atum) a11y #34768

merged 1 commit into from
Jul 21, 2021

Conversation

brianteeman
Copy link
Contributor

When an image is used for functional purposes eg a link then then alt text should describe the function and not the image. Otherwise the link may appear to be empty to assistive technology and therefore ignored.

https://www.w3.org/WAI/tutorials/images/functional/#image-used-alone-as-a-linked-logo

In the atum template the logo in the top right is sometimes used as a link to the homepage. So in those circumstance it needs an alt text that describe where the link takes you that and not an aria label on the image.

When an image is used for functional purposes eg a link then then alt text should describe the function and not the image. Otherwise the link may appear to be empty to assistive technology and therefore ignored.

https://www.w3.org/WAI/tutorials/images/functional/#image-used-alone-as-a-linked-logo

In the atum template the logo in the top right is sometimes used as a link to the homepage. So in those circumstance it needs an alt text that describe where the link takes you that and not an aria label on the image.
@Quy
Copy link
Contributor

Quy commented Jul 14, 2021

I have tested this item ✅ successfully on 23e87bf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34768.

@Quy Quy added PR-4.0-dev a11y Accessibility labels Jul 14, 2021
@hans2103
Copy link
Contributor

I have tested this item ✅ successfully on 23e87bf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34768.

@alikon
Copy link
Contributor

alikon commented Jul 16, 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34768.

@joomla-cms-bot joomla-cms-bot added RTC This Pull Request is Ready To Commit and removed a11y Accessibility labels Jul 16, 2021
@Quy Quy added this to the Joomla 4.0 milestone Jul 16, 2021
@Quy Quy added the a11y Accessibility label Jul 20, 2021
@chmst chmst merged commit d746df3 into joomla:4.0-dev Jul 21, 2021
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 21, 2021
@chmst
Copy link
Contributor

chmst commented Jul 21, 2021

Thanks!

@brianteeman
Copy link
Contributor Author

thank you

@brianteeman brianteeman deleted the a11yhome branch July 21, 2021 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants