[Bug 9389] New: Accessibility - "no more renewals" alert box difficult to access for screen-reader users
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Bug ID: 9389 Summary: Accessibility - "no more renewals" alert box difficult to access for screen-reader users Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Staff Client Assignee: koha-bugs@lists.koha-community.org Reporter: bart.tj@gmail.com CC: gmcharlt@gmail.com When you reach the maximum number of times a book can be renewed in regards to the settings of circulation settings the alert box is difficult to read when using screen reader software and also difficult to navigate to as it is in plain text and in a list which is also reasonably far down the html page in most cases. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Bart Jorgensen <bart.tj@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|koha-bugs@lists.koha-commun |bart.tj@gmail.com |ity.org | --- Comment #1 from Bart Jorgensen <bart.tj@gmail.com> --- Created attachment 14575 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14575&action=edit Added header tags to error message to make it more accessible -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Liz Rea <wizzyrea@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14575|0 |1 is obsolete| | --- Comment #2 from Liz Rea <wizzyrea@gmail.com> --- Created attachment 14582 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14582&action=edit Bug 9389 added header tags to error message to test: -search for a patron -issue a book that cannot be renewed (dependant on circulation rules) -error should show with error message "No more renewals possible," but be wrapped in a <h5> Signed-off-by: Liz Rea <liz@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Liz Rea (CatalystIT) <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |liz@catalyst.net.nz -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #3 from Owen Leonard <oleonard@myacpl.org> --- This change doesn't really make sense. In the template we have a lot of possible errors. Here are a few: [% IF ( DEBARRED ) %] <li>Patron is restricted</li> [% END %] [% IF ( NO_MORE_RENEWALS ) %] <h5><li>No more renewals possible</li></h5> [% END %] [%IF ( AGE_RESTRICTION ) %] <li>Age restriction [% AGE_RESTRICTION %].</li> [% END %] [% IF ( EXPIRED ) %] <li>Patron's card is expired</li> [% END %] 1. <h5> inside an <li> is not valid XHTML. 2. It doesn't make sense to wrap only one of many possible error messages in <h5> It would make more sense, I think, to add a heading to the message box itself: <div id="circ_impossible" class="dialog alert"> <!-- RESULT OF ISSUING REQUEST --> <h5>Could not complete checkout</h5> ...and have the list of messages below. Would this fulfill the same requirements? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Bart Jorgensen <bart.tj@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14582|0 |1 is obsolete| | --- Comment #4 from Bart Jorgensen <bart.tj@gmail.com> --- Created attachment 14608 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14608&action=edit Moved header tags to improve patch I've made the changes suggested by Owen - moved header tags to cover full range of options. (This also obsoletes my patch on Bug 9390 which is very similar) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9389 Liz Rea (CatalystIT) <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |RESOLVED Resolution|--- |WONTFIX -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org