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

Possible Accessibility Compliance #12

Open
dbltoe opened this issue Jul 26, 2022 · 0 comments
Open

Possible Accessibility Compliance #12

dbltoe opened this issue Jul 26, 2022 · 0 comments

Comments

@dbltoe
Copy link

dbltoe commented Jul 26, 2022

I'm working on the same thing in 1.5.8 but really like to use this mod instead of the norm and think it should be built-in.

Because of the close proximity of the Image and the Title, using the Title of the item for an alt tag results in a minor hit for accessibility.

With the main information being in the Title of the item, I think accessibility can be better met by making the alt tag for the image blank and setting the role to presentation.

<img src="images/product1.jpg" alt="" role="presentation" title="Product 1" width="100" height="75"

I use the example with both the empty alt and role setting as this would make it compatible with all browsers and readers.

I wish I were able to submit proposed fille changes but, it's just not in my level of expertise. Hopefully, this will explain what might be done to make the mod even better.

While some would remove the H and W for CSS, this also tells the browser that the item is presentational and is ALWAYS overwritten by any CSS setting(s) made.

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

No branches or pull requests

1 participant