Bugzilla – Attachment 183694 Details for
Bug 40283
Accessibility of the status message in the OPAC
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 40283: Your summary page.
Bug-40283-Your-summary-page.patch (text/plain), 5.09 KB, created by
Owen Leonard
on 2025-07-02 12:40:52 UTC
(
hide
)
Description:
Bug 40283: Your summary page.
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2025-07-02 12:40:52 UTC
Size:
5.09 KB
patch
obsolete
>From d1439d9cd91de0c13a77a4b8ec948ce271462f8f Mon Sep 17 00:00:00 2001 >From: nina martinez <nina.martinez@biblibre.com> >Date: Wed, 11 Jun 2025 17:19:48 +0200 >Subject: [PATCH] Bug 40283: Your summary page. > >This patch add role="status" and role="alert" to the Summary page. >(opac-user) > >Test plan: > >1- Enable the system preference OpacReportProblem. If you have a > checkout record, add a note. A message should appear: "Your note > about ...". Inspect the html code and notice that there is a > role="status" in the message tag. >2- Create a new patron account with a card expiration date set to > tomorrow. Go to the OPAC, log in to this patron account? You should > notice on your summary page a message: "Your library card will expire > on ...". Inspect the html code and notice that there is a > role="alert" in the message tag. >3- Go to the staff interface, patron details, and change the card > expiration date to yesterday. Go back to the OPAC and you should now > have the message: "Your account has expired as of...". >4- Go to the staff interface, choose your patron, click on more, and > choose discharge. >5- Go back to the OPAC, log in to this patron account, and a message > should appear: "Your account is frozen because it has been > discharged." Inspect the HTML code and notice that there is a > role='alert' in the message tag. > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> >--- > .../opac-tmpl/bootstrap/en/modules/opac-user.tt | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >index 1c63a78d035..8f0ab237deb 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >@@ -60,10 +60,12 @@ > <a href="/cgi-bin/koha/opac-main.pl?logout.x=1" class="d-print-none">Click here if you're not [% INCLUDE 'patron-title.inc' patron = logged_in_user %]</a></p > > > >- [% IF ( patronupdate ) %]<div class="alert alert-info"><h2>Thank you!</h2><p>Your corrections have been submitted to the library, and a staff member will update your record as soon as possible.</p></div>[% END %] >+ [% IF ( patronupdate ) %] >+ <div class="alert alert-info" role="status"><h2>Thank you!</h2><p>Your corrections have been submitted to the library, and a staff member will update your record as soon as possible.</p></div> >+ [% END %] > > [% IF failed_holds %] >- <div class="alert alert-info"> >+ <div class="alert alert-info" role="status"> > <h2>Notice:</h2> > <p>One or more holds were not placed due to following errors:</p> > <ul> >@@ -120,14 +122,14 @@ > [% END %] > > [% IF ( borrower.warndeparture ) %] >- <div class="alert alert-warning" id="warndeparture"> >+ <div class="alert alert-warning" id="warndeparture" role="alert"> > <strong>Please note:</strong><span> Your library card will expire on <span id="warndeparture_date">[% borrower.warndeparture | $KohaDates %]</span>. Please contact the library for more information.</span> > [% IF ( borrower.returnbeforeexpiry ) %]<span id="warndeparture_returnbeforeexpiry"> Also note that you must return all checked out items before your library card expires.</span>[% END %] > </div> > [% END %] > > [% IF ( borrower.warnexpired ) %] >- <div class="alert alert-warning" id="warnexpired"> >+ <div class="alert alert-warning" id="warnexpired" role="alert"> > <strong>Please note: </strong><span>Your account has expired as of [% borrower.warnexpired | $KohaDates %]. Please contact the library if you wish to renew your account.</span> > </div> > [% END %] >@@ -161,7 +163,7 @@ > [% END %] > > [% IF ( patron_flagged ) %] >- <div class="alert alert-warning"> >+ <div class="alert alert-warning" role="alert"> > <ul> > [% IF ( userdebarred ) %] > [% IF ( discharge_available ) %] >@@ -196,7 +198,7 @@ > <div class="alert alert-info" id="savings"> Congratulations, you have saved a total of [% savings | $Price with_symbol => 1 %] by using the library. </div> > [% END %] > >- <div class="alert alert-info" id="notesaved" style="display:none;"></div> >+ <div class="alert alert-info" id="notesaved" style="display:none;" role="status"></div> > [% IF ( OpacMySummaryNote ) %] > [% PROCESS koha_news_block news => OpacMySummaryNote %] > [% END %] >-- >2.39.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 40283
:
183665
|
183666
|
183667
|
183668
|
183669
|
183670
|
183671
|
183672
|
183673
|
183674
|
183683
|
183684
|
183685
|
183686
|
183689
|
183690
|
183691
|
183692
|
183693
| 183694 |
183695
|
183696
|
183697
|
183698
|
183699
|
183700
|
183701