search

IUBLibTech / newton_chymistry

New version of 'The Chymistry of Isaac Newton', using XProc pipelines to generate a website based on TEI XML encodings of Newton's alchemical manuscripts, and Apache Solr as a search engine.
2 stars 0 forks source link

Image missing a text alternative (IU Accessibility Mandate) #139

Open mdalmau opened 6 months ago

mdalmau commented 6 months ago

21 occurrences : HTML pages with images need an "alt" attribute:

Network Workbench logo

IU has a mandate that all web sites published on IU servers must meet WCAG I and II Accessibility requirements. If the requirements are NOT met, sites will be taken down. To learn more about this initiative, visit the Digital Strategy Initiative web site.

mdalmau commented 6 months ago

@tubesoft : At the Newton meeting today, Bill said you would be working on these accessibility issues. I requested SiteImprove access for you. Once you have access, you will login using your @iu credentials. If you want to read more about SIteImprove and accessibility, LOGIN to the KB and start here.

Could you have a look at this issue and confirm it's something you can fix? Once you confirm, we can figure out a testing workflow.

tubesoft commented 6 months ago

@mdalmau Some of the occurrences are LSA pages. Should we pass those to @wehooper ? So far, I am going to work on non-LSA occurrences.

tubesoft commented 6 months ago

@mdalmau, I fixed the issues and committed, as you can see above.

mdalmau commented 6 months ago

@tubesoft : I am slightly worried that we will create a conflict when we merge since I am assuming a lot of the fixes you made to the rendition issues in that pending pull request are in the same files for the accessibility pull request. Can you undo commits until we get the first pull request merged and dev/production updated?

tubesoft commented 6 months ago

@mdalmau I committed to a different branch from rendition issues, so merging the pending request does not affect changes for the accessibility.

mdalmau commented 5 months ago

@tubesoft I am not sure how to handle this in GitHub since you fixed the non-LSA pages. Maybe I should have made it into two issues? I won't close this one for now.