-
Notifications
You must be signed in to change notification settings - Fork 433
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
Minor Accessibility Fixes & Enable accessibility scan on more pages #2468
Conversation
…xclusion from header
…o prove it now passes
The changes from Before and after: More seriously, the date slider doesn't work. It seems to be fatally bugged. Here's a recording: date-slider.mp4Alternate link for video since it's not showing for me here: https://imgur.com/a/wwzuJqk Before testing I removed my |
@alanorth : Thanks for your testing! Both those issues sound like browser caching issues. I forgot to mention that this PR changes CSS, which means you will need to do a hard refresh/reload in your browser (Ctrl+Reload button) after installing the PR. Our UI / Angular will cache CSS files for a period of time... so if you attempt to run the PR with old CSS, you will get odd behavior. The The slider issues are also CSS. I saw these briefly myself as our If you have a chance to re-test, I think a hard refresh will solve both issues. (I should also note this sort of caching can occur with any PR which changes CSS. It's not unique to this PR, so if you are testing other PRs that modify CSS, it's good to get in the habit of doing a hard refresh anytime you notice something odd... it could be just that your browser has cached the old CSS.) |
@tdonohue ah right! A shift-refresh fixed the broken date slider, and the The community list, item view, and search pages are all working as expected for me now. |
Successfully created backport PR for |
Description
Minor dependency updates to support accessibility fixes:
axe-core
(used for accessibility scans in e2e tests)ng2-nouislider
andnouislider
(both used for date filter search slider) to fix accessibility issuesAccessibility issues fixed:
aria-label
values to slider handlebars. Fix is verified by changes tomy-dspace.cy.ts
andsearch-page.cy.ts
as we no longer have to exclude thenouislider
from accessibility tests.my-dspace.cy.ts
andsearch-page.cy.ts
/community-list
page - verified by changes tocommunity-list.cy.ts
<h5>
tags were being used based on their style even though anh2
orh4
provided better heading ordering.Instructions for Reviewers
/community-list
)/items/[uuid]
)/search
or/mydspace