Bug 26039 added some text intended only for screen reader users. Since the Bootstrap 4 upgrade this text is shown because the CSS class is obsolete. I propose to replace the use of the "hidden" class with "sr-only." https://getbootstrap.com/docs/4.5/utilities/screen-readers/
Created attachment 109829 [details] [review] Bug 26421: Use Bootstrap screen reader text class for shelf browser messages This patch udpates the markup added by Bug 26039, replacing the "hidden" class, which is no longer functional with Bootstrap 4, with "sr-only," a class designed specifically for screen reader text. The patch also corrects a typo, "shelfbrowers" -> "shelf browser" To test, apply the patch and make sure the OPACShelfBrowser system preference is enabled. View the detail page for a bibliographic record with items. In the holdings table, you should see links labeled "Browse shelf." When you open the shelf browser there should be a link at the top that reads "Close shelf browser." If you have access to a screen reader, test that the hidden text is announced by the reader. "Browse shelf (Opens below)" and "Close shelf browser (Hides shelf browser)."
*** Bug 26446 has been marked as a duplicate of this bug. ***
Created attachment 110058 [details] [review] Bug 26421: Use Bootstrap screen reader text class for shelf browser messages This patch udpates the markup added by Bug 26039, replacing the "hidden" class, which is no longer functional with Bootstrap 4, with "sr-only," a class designed specifically for screen reader text. The patch also corrects a typo, "shelfbrowers" -> "shelf browser" To test, apply the patch and make sure the OPACShelfBrowser system preference is enabled. View the detail page for a bibliographic record with items. In the holdings table, you should see links labeled "Browse shelf." When you open the shelf browser there should be a link at the top that reads "Close shelf browser." If you have access to a screen reader, test that the hidden text is announced by the reader. "Browse shelf (Opens below)" and "Close shelf browser (Hides shelf browser)." Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Screen reader announced the text as expected.
Created attachment 110117 [details] [review] Bug 26421: Use Bootstrap screen reader text class for shelf browser messages This patch udpates the markup added by Bug 26039, replacing the "hidden" class, which is no longer functional with Bootstrap 4, with "sr-only," a class designed specifically for screen reader text. The patch also corrects a typo, "shelfbrowers" -> "shelf browser" To test, apply the patch and make sure the OPACShelfBrowser system preference is enabled. View the detail page for a bibliographic record with items. In the holdings table, you should see links labeled "Browse shelf." When you open the shelf browser there should be a link at the top that reads "Close shelf browser." If you have access to a screen reader, test that the hidden text is announced by the reader. "Browse shelf (Opens below)" and "Close shelf browser (Hides shelf browser)." Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 20.11, thanks to everybody involved!
missing dependencies for 20.05.x, no backport