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

Container element is empty (IU Accessibility Mandate) #133

Closed mdalmau closed 5 months ago

mdalmau commented 6 months ago

20 occurrences of this issue ... Looks like the HTML is spitting out an empty unordered list in the search / search results pages: <ul class="results"></ul>


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

In solr-response-to-html.xsl, the search results are printed between <ul class="results"> and </ul>. That is, if the search results are none, only <ul class="results"></ul> remains (See image below). From the perspective of programming, it does not seem to be too bad. In order to omit <ul class="results"></ul>, I would need to add a condition of printing <ul class="results"></ul>. Do we really need to follow this warning strictly? Program-wise, leaving it as it is is much simpler.

image
mdalmau commented 6 months ago

@tubesoft : yes, all these issues must be fixed. There’s no way around this. If you think it’s beyond your comfort level, we can set that aside for Bill to figure out with Conal but anything you can do, please do because we are under a deadline! Thank you.

tubesoft commented 6 months ago

Okay, I managed to fix it. Before:

image

After:

image
mdalmau commented 5 months ago

Since this issue is now part of https://github.com/IUBLibTech/newton_chymistry/pull/146, I will mark this issue as "done" so I can better see the remaining accessibility issues!