https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40283
Aleisha Amohia <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] QA Contact|[email protected] |[email protected] |y.org | --- Comment #30 from Aleisha Amohia <[email protected]> --- Hi Nina, thanks for this patchset! A question about the role parameter - should it be added to **every** message that might display for the user? For example, when I was reviewing the changes to the discharge page, I noticed a few <p> messages that have not been given a role: +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-discharge.tt @@ -37,7 +37,7 @@ [% END %] [% IF success %] - <p>Your discharge request has been sent. Your discharge will be available on this page within a few days.</p> + <p role="status">Your discharge request has been sent. Your discharge will be available on this page within a few days.</p> [% ELSIF available %] <p><a href="/cgi-bin/koha/opac-discharge.pl?op=get">Get your discharge</a></p> [% ELSIF pending %] @@ -59,7 +59,7 @@ [% IF failure %] <p> There was an error during the discharge process </p> [% END %] Specifically the "Get your discharge" link and "There was an error during the discharge process" failure message. Should these kinds of messages have roles too? -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
