Bugzilla – Attachment 151594 Details for
Bug 33812
Accessibility: OPAC messaging preferences is missing form labels
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 33812: Checkboxes need labels in opac-messaging.tt
Bug-33812-Checkboxes-need-labels-in-opac-messaging.patch (text/plain), 9.60 KB, created by
Matt Blenkinsop
on 2023-05-23 16:10:18 UTC
(
hide
)
Description:
Bug 33812: Checkboxes need labels in opac-messaging.tt
Filename:
MIME Type:
Creator:
Matt Blenkinsop
Created:
2023-05-23 16:10:18 UTC
Size:
9.60 KB
patch
obsolete
>From 7393ee77035626745dedc883bdb8ee63deac0698 Mon Sep 17 00:00:00 2001 >From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> >Date: Tue, 23 May 2023 16:08:31 +0000 >Subject: [PATCH] Bug 33812: Checkboxes need labels in opac-messaging.tt > >Aria-labels have been added to checkboxes in the OPAC messaging preferences table to assist with screen reading. The relevant preference has been mentioned in each label to identify what the checkbox is for. >--- > .../bootstrap/en/modules/opac-messaging.tt | 18 +++++++++++------- > 1 file changed, 11 insertions(+), 7 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >index 863e56664b..21ae6c9f2e 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >@@ -80,7 +80,7 @@ > [% ELSIF ( messaging_preference.Auto_Renewals ) %]<span>Auto renewal</span> > [% ELSE %]<span>Unknown</span> [% END %]</td> > [% IF ( messaging_preference.takes_days ) %] >- <td><select class="input-mini" name="[% messaging_preference.message_attribute_id | html %]-DAYS"> >+ <td><select class="input-mini" name="[% messaging_preference.message_attribute_id | html %]-DAYS" aria-label="Choose how many days in advance you wish to receive notice"> > [% FOREACH select_day IN messaging_preference.select_days %] > [% IF ( select_day.selected ) %] > <option value="[% select_day.day | html %]" selected="selected">[% select_day.day | html %]</option> >@@ -96,9 +96,9 @@ > [% IF ( messaging_preference.transport_sms ) %] > <td class="selectcol"> > [% IF ( messaging_preference.transports_sms ) %] >- <input type="checkbox" class="pmp_sms" id="sms[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="sms" checked="checked" /> >+ <input type="checkbox" aria-label="Unselect sms messaging" class="pmp_sms" id="sms[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="sms" checked="checked" /> > [% ELSE %] >- <input type="checkbox" class="pmp_sms" id="sms[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="sms" /> >+ <input type="checkbox" aria-label="Select sms messaging" class="pmp_sms" id="sms[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="sms" /> > [% END %] > </td> > [% ELSE %] >@@ -110,12 +110,14 @@ > <td class="selectcol"> > [% IF ( messaging_preference.transports_phone ) %] > <input type="checkbox" >+ aria-label="Unselect notification by phone" > class="pmp_phone" > id="phone[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" > value="phone" checked="checked" /> > [% ELSE %] > <input type="checkbox" >+ aria-label="Select notification by phone" > class="pmp_phone" > id="phone[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" >@@ -131,12 +133,14 @@ > <td class="selectcol"> > [% IF ( messaging_preference.transports_itiva ) %] > <input type="checkbox" >+ aria-label="Unselect itiva messaging" > class="itiva" > id="itiva[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" > value="itiva" checked="checked" /> > [% ELSE %] > <input type="checkbox" >+ aria-label="Select notification by phone" > class="itiva" > id="itiva[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" >@@ -151,9 +155,9 @@ > [% IF ( messaging_preference.transport_email ) %] > <td class="selectcol"> > [% IF ( messaging_preference.transports_email ) %] >- <input type="checkbox" class="pmp_email" id="email[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="email" checked="checked" /> >+ <input type="checkbox" aria-label="Unselect email messaging"class="pmp_email" id="email[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="email" checked="checked" /> > [% ELSE %] >- <input type="checkbox" class="pmp_email" id="email[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="email" /> >+ <input type="checkbox" aria-label="Select email messaging" class="pmp_email" id="email[% messaging_preference.message_attribute_id | html %]" name="[% messaging_preference.message_attribute_id | html %]" value="email" /> > [% END %] > </td> > [% ELSE %] >@@ -163,9 +167,9 @@ > [% IF ( messaging_preference.has_digest ) %] > <td class="selectcol"> > [% IF ( messaging_preference.digest ) %] >- <input type="checkbox" id="digest[% messaging_preference.message_attribute_id | html %]" value="[% messaging_preference.message_attribute_id | html %]" name="digest" checked="checked" data-toggle="tooltip" title="You must select a digestible transport to select digests" /> >+ <input type="checkbox" aria-label="Unselect digests" id="digest[% messaging_preference.message_attribute_id | html %]" value="[% messaging_preference.message_attribute_id | html %]" name="digest" checked="checked" data-toggle="tooltip" title="You must select a digestible transport to select digests" /> > [% ELSE %] >- <input type="checkbox" id="digest[% messaging_preference.message_attribute_id | html %]" value="[% messaging_preference.message_attribute_id | html %]" name="digest" data-toggle="tooltip" title="You must select a digestible transport to select digests" /> >+ <input type="checkbox" aria-label="Select digests"id="digest[% messaging_preference.message_attribute_id | html %]" value="[% messaging_preference.message_attribute_id | html %]" name="digest" data-toggle="tooltip" title="You must select a digestible transport to select digests" /> > [% END %] > </td> > [% ELSE %] >-- >2.37.1 (Apple Git-137.1)
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 33812
:
151594
|
154320
|
156372