-
Notifications
You must be signed in to change notification settings - Fork 7
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
Investigate options and evaluate effort to replace blacklight_heatmaps gem #2611
Comments
I think this is difficult to fully evaluate in terms of effort until we have a design for what an accessible results list looks like, and possibly how that interacts with the map. I think that's the initial key factor here. Here's a summary of my understanding of #490:
I think #504 is out of scope for this issue because it only pertains to the item level view? The tricky a11y concerns from #1813 are all about the search results list. As Jack pointed out, leaflet has keyboard navigation. I think the concerns are clearly listed in that issue, so I won't duplicate them here. Existing SolutionsBlacklight Heatmapshttps://github.com/sul-dlss/blacklight_heatmaps Pros:
Cons:
Blacklight::Mapshttps://github.com/projectblacklight/blacklight-maps Pros:
Cons:
Write an Exhibits specific solution with Leaflet.markerclusterPros:
Cons:
|
The existing blacklight_heatmaps gem has both accessibility issues and an accumulation of feature requests/suggestions:
Would need significant accessibility design and technical work to improve the gem. Nick pointed out more recent plugins that could be swapped in.
From Alan's notes:
The goal of this issue is to investigate these options, evalute pros/cons and level of effort.
The text was updated successfully, but these errors were encountered: