[open-ils-commits] [GIT] Evergreen ILS branch rel_3_2 updated. 3cc7e288643b3188ff5f6899b8cf3e559d74cec7
Evergreen Git
git at git.evergreen-ils.org
Mon Aug 5 21:12:27 EDT 2019
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".
The branch, rel_3_2 has been updated
via 3cc7e288643b3188ff5f6899b8cf3e559d74cec7 (commit)
from 334e8b9d3a03c4293a4d3d138c8af9fbd5cc3a14 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 3cc7e288643b3188ff5f6899b8cf3e559d74cec7
Author: Jane Sandberg <sandbej at linnbenton.edu>
Date: Sat Nov 10 12:32:04 2018 -0800
LP1735768: adding labels to inputs in the public catalog
Signed-off-by: Jane Sandberg <sandbej at linnbenton.edu>
Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
diff --git a/Open-ILS/src/templates/opac/browse.tt2 b/Open-ILS/src/templates/opac/browse.tt2
index dbaaa35886..21c3e65a3f 100644
--- a/Open-ILS/src/templates/opac/browse.tt2
+++ b/Open-ILS/src/templates/opac/browse.tt2
@@ -52,6 +52,7 @@
id="browse-search-class" browse_only=1 plural=1 %]
[% control_bterm = BLOCK %]<input type="text" name="bterm" id="browse-term"
+ aria-label="[% l('Browse term') %]"
value="[% CGI.param('bterm') | html %]" />[% END %]
[% control_locg = INCLUDE build_org_selector id='browse-context'
show_loc_groups=1
diff --git a/Open-ILS/src/templates/opac/myopac/update_email.tt2 b/Open-ILS/src/templates/opac/myopac/update_email.tt2
index a729953248..34e4420542 100644
--- a/Open-ILS/src/templates/opac/myopac/update_email.tt2
+++ b/Open-ILS/src/templates/opac/myopac/update_email.tt2
@@ -28,8 +28,8 @@
[% END %]
<table>
<tr><td>[% l('Current Email') %]</td><td>[% ctx.user.email | html %]</td></tr>
- <tr><td>[% l('Current Password') %]</td><td><input type='password' name='current_pw'/></td></tr>
- <tr><td>[% l('New Email') %]</td><td><input type='text' name='email' value='[% ctx.invalid_email | html %]'/></td></tr>
+ <tr><td><label for="current_pw">[% l('Current Password') %]</label></td><td><input type='password' name='current_pw' id='current_pw' /></td></tr>
+ <tr><td><label for="new_email">[% l('New Email') %]</label></td><td><input type='text' name='email' id='new_email' value='[% ctx.invalid_email | html %]'/></td></tr>
<tr><td colspan='2' align='center'><input value="[% l('Submit') %]" type='submit'/></td></tr>
</table>
</form>
diff --git a/Open-ILS/src/templates/opac/myopac/update_password.tt2 b/Open-ILS/src/templates/opac/myopac/update_password.tt2
index bef8b994c6..d86d8e1cee 100644
--- a/Open-ILS/src/templates/opac/myopac/update_password.tt2
+++ b/Open-ILS/src/templates/opac/myopac/update_password.tt2
@@ -30,9 +30,9 @@
<form method='post' id='account-update-email' autocomplete='off'>
<table>
- <tr><td>[% l('Current Password') %]</td><td><input type='password' name='current_pw'/></td></tr>
- <tr><td>[% l('New Password') %]</td><td><input type='password' name='new_pw'/></td></tr>
- <tr><td>[% l('New Password Again') %]</td><td><input type='password' name='new_pw2'/></td></tr>
+ <tr><td><label for="current_pw">[% l('Current Password') %]</label></td><td><input type='password' name='current_pw' id='current_pw' /></td></tr>
+ <tr><td><label for="new_pw">[% l('New Password') %]</label></td><td><input type='password' name='new_pw' id='new_pw' /></td></tr>
+ <tr><td><label for="new_pw2">[% l('New Password Again') %]</label></td><td><input type='password' name='new_pw2'id='new_pw2' /></td></tr>
<tr><td colspan='2' align='center'><input value="[% l('Submit') %]" type='submit'/></td></tr>
</table>
</form>
diff --git a/Open-ILS/src/templates/opac/parts/advanced/global_row.tt2 b/Open-ILS/src/templates/opac/parts/advanced/global_row.tt2
index 59b8c4e5e1..32a550abb5 100644
--- a/Open-ILS/src/templates/opac/parts/advanced/global_row.tt2
+++ b/Open-ILS/src/templates/opac/parts/advanced/global_row.tt2
@@ -45,7 +45,7 @@
<option value="[% o.value %]" [% c == o.value ? ' selected="selected"' : '' %]>[% o.label %]</option>
[% END %]
</select>
- <input title="[% l('Search term') %]"
+ <input title="[% l('Search term') %]" aria-label="[% l('Search term') %]"
type='text' size='18' name='query' value="[% q | html %]" x-webkit-speech [% IF loop.index == 0 %] autofocus [% END %] />
<a href="javascript:;" class="row-remover"
title="[% l('Remove row') %]" alt="[% l('Remove row') %]"
diff --git a/Open-ILS/src/templates/opac/parts/advanced/numeric.tt2 b/Open-ILS/src/templates/opac/parts/advanced/numeric.tt2
index f768c10202..3339f3aa2a 100644
--- a/Open-ILS/src/templates/opac/parts/advanced/numeric.tt2
+++ b/Open-ILS/src/templates/opac/parts/advanced/numeric.tt2
@@ -16,7 +16,7 @@
<option value="item_barcode">[% l('Item Barcode') %]</option>
</select>
- <input type="text" name="query" size="16" autofocus placeholder='[% l("Identifier") %]' />
+ <input type="text" name="query" size="16" autofocus placeholder='[% l("Identifier") %]' aria-label='[% l("Identifier") %]' />
<br/>
[%- lib_select_id="numeric_search_library" -%]
<label for="[% lib_select_id %]"><strong>[% l("Search Library:") %]</strong>
diff --git a/Open-ILS/src/templates/opac/parts/advanced/search.tt2 b/Open-ILS/src/templates/opac/parts/advanced/search.tt2
index 1ac3b67815..20c9caaa3f 100644
--- a/Open-ILS/src/templates/opac/parts/advanced/search.tt2
+++ b/Open-ILS/src/templates/opac/parts/advanced/search.tt2
@@ -96,10 +96,10 @@
<br/>
<div id='adv_special_block'>
[% END %]
- <input title="[% l('Search date') %]"
+ <input title="[% l('Search date') %]" aria-label="[% l('Search date') %]"
name='date1' type='text' size='4' maxlength='4' value="[% CGI.param('date1') | html %]" />
<span id='adv_global_pub_date_2_span' class='[% CGI.param("pubdate") == "between" ? "" : "hide_me" %]'>
- [% l("and") %] <input name='date2'
+ [% l("and") %] <input name='date2' aria-label="[% l('Secondary search date') %]"
title="[% l('Secondary search date') %]" type='text' size='4' maxlength='4' value="[% CGI.param('date2') | html %]" />
</span>
[% IF !ctx.exclude_electronic_checkbox %]
diff --git a/Open-ILS/src/templates/opac/parts/anon_list.tt2 b/Open-ILS/src/templates/opac/parts/anon_list.tt2
index 56fbd6cee5..ed0b54624a 100644
--- a/Open-ILS/src/templates/opac/parts/anon_list.tt2
+++ b/Open-ILS/src/templates/opac/parts/anon_list.tt2
@@ -16,7 +16,7 @@
<input type="hidden" name="orig_referrer" value="[% CGI.referer | html %]" />
<input type="hidden" name="redirect_to" value="[% mkurl('', {}, ['list_none_selected', 'cart_none_selected']) %]" />
<div class="bbag-action" style="clear:both;">
- <select name="action">
+ <select name="action" aria-label="[% l('Actions for these items') %]">
<option>[% l('-- Actions for these items --') %]</option>
<option value="place_hold">[% l('Place hold') %]</option>
<option value="print">[% l('Print title details') %]</option>
@@ -33,7 +33,7 @@
</select>
[%- INCLUDE "opac/parts/preserve_params.tt2"; %]
<input class="opac-button" type="submit" value="[% l('Go') %]" />
- <input type="checkbox" name="clear_cart">[% l('Clear entire basket when action complete') %]</input>
+ <input type="checkbox" name="clear_cart" id="clear_cart" /><label for="clear_cart">[% l('Clear entire basket when action complete') %]</label>
[% IF CGI.param('cart_none_selected') %]
<span class="error">[% l('No items were selected') %]</span>
[% END %]
@@ -43,7 +43,8 @@
<thead id="acct_list_header_anon">
<tr>
<td class='list_checkbox'>
- <input type="checkbox" checked="checked" onclick="
+ <input type="checkbox" checked="checked"
+ aria-label="[% l('Select all records') %]" onclick="
var inputs=document.getElementsByTagName('input');
for (i = 0; i < inputs.length; i++) {
if (inputs[i].name == 'record' && !inputs[i].disabled) inputs[i].checked = this.checked;}"/>
@@ -59,7 +60,7 @@
PROCESS get_marc_attrs args=attrs %]
<tr>
<td class="list_checkbox">
- <input type="checkbox" checked="checked" name="record" value="[% item %]" />
+ <input type="checkbox" checked="checked" name="record" value="[% item %]" aria-label="[% l('Select record') %]" />
</td>
<td class="list_entry" data-label="[% l('Title') %]"><a href="[% mkurl(ctx.opac_root _ '/record/' _ item, {}, ['edit_notes', 'id']) %]">[% attrs.title | html %]</a></td>
<td class="list_entry" data-label="[% l('Author(s)') %]"><a href="[%-
diff --git a/Open-ILS/src/templates/opac/parts/cart.tt2 b/Open-ILS/src/templates/opac/parts/cart.tt2
index ae587bcb80..84514133f5 100644
--- a/Open-ILS/src/templates/opac/parts/cart.tt2
+++ b/Open-ILS/src/templates/opac/parts/cart.tt2
@@ -1,6 +1,6 @@
<div id="record_basket">
<div id="basket_actions">
- <select id="select_basket_action">
+ <select id="select_basket_action" aria-label="[% l('Select basket action') %]">
<option value="">[% l('-- Basket Actions --') %]</option>
<option value="[% mkurl(ctx.opac_root _ '/mylist', {}) %]">[% l('View Basket') %]</option>
<option value="[% mkurl(ctx.opac_root _ '/mylist/move', { action => 'place_hold', entire_list => 1 }) %]">[% l('Place Holds') %]</option>
diff --git a/Open-ILS/src/templates/opac/parts/login/form.tt2 b/Open-ILS/src/templates/opac/parts/login/form.tt2
index 3146d5bcd6..fa391f0ff0 100644
--- a/Open-ILS/src/templates/opac/parts/login/form.tt2
+++ b/Open-ILS/src/templates/opac/parts/login/form.tt2
@@ -50,7 +50,7 @@
%]
<input type='hidden' name='redirect_to' value='[% redirect | html %]'/>
<input type="checkbox" name="persist" id="login_persist" /><label for="login_persist"> [% l('Stay logged in?') %]</label>
- <input type="submit" value="[% l('Log in') %]" alt="[% l('Log in') %]" class="opac-button" />
+ <input type="submit" value="[% l('Log in') %]" class="opac-button" />
</div>
<input id="client_tz_id" name="client_tz" type="hidden" />
</form>
diff --git a/Open-ILS/src/templates/opac/parts/place_hold.tt2 b/Open-ILS/src/templates/opac/parts/place_hold.tt2
index c4e7e25a37..f2d1bba3bc 100644
--- a/Open-ILS/src/templates/opac/parts/place_hold.tt2
+++ b/Open-ILS/src/templates/opac/parts/place_hold.tt2
@@ -105,7 +105,8 @@ function maybeToggleNumCopies(obj) {
<label for="hold_usr_is_requestor_not">
[% l("Place hold for patron by barcode:") %]
</label>
- <input type="text" name="hold_usr" id="hold_usr_input"
+ <input type="text" name="hold_usr" id="hold_usr_input"
+ aria-label="[% l('Barcode') %]"
value="[% usr_barcode | html %]"
onchange="staff_hold_usr_barcode_changed();"
onpaste="setTimeout(staff_hold_usr_barcode_changed,1);"
@@ -260,11 +261,13 @@ function maybeToggleNumCopies(obj) {
[% END %]
</blockquote>
<blockquote>
+ <label for="hold_suspend">
[% IF ctx.hold_data.size == 1;
l('Suspend this hold?');
ELSE;
l('Suspend these holds?');
END %]
+ </label>
<img src="[% ctx.media_prefix %]/images/question-mark.png[% ctx.cache_key %]"
alt="[% l('Suspend Hold Help') %]"
title="[% l('A suspended hold will retain its place in the queue, but will not be fulfilled until it has been activated.') %]" />
@@ -279,7 +282,7 @@ function maybeToggleNumCopies(obj) {
</blockquote>
</p>
[% IF CGI.param('from_basket') %]
- <blockquote><input type="checkbox" name="clear_cart">[% l('Clear basket after holds are requested?') %]</input></blockquote>
+ <blockquote><input type="checkbox" name="clear_cart" id="clear_cart" /><label for="clear_cart">[% l('Clear basket after holds are requested?') %]</label></blockquote>
[% END %]
<input id="place_hold_submit" type="submit" name="submit"
value="[% l('Submit') %]" title="[% l('Submit') %]"
diff --git a/Open-ILS/src/templates/opac/parts/result/table.tt2 b/Open-ILS/src/templates/opac/parts/result/table.tt2
index 63b4980360..21652c4069 100644
--- a/Open-ILS/src/templates/opac/parts/result/table.tt2
+++ b/Open-ILS/src/templates/opac/parts/result/table.tt2
@@ -108,6 +108,7 @@
<input type="checkbox" id="select-[% rec.bre_id %]" name="selected_record"
[% IF is_selected %] checked="checked" [% END %]
title="[% l('Add to Basket') %]"
+ aria-label="[% l('Add to Basket') %]"
class="result_record_selector hidden" value="[% rec.bre_id %]"></input>
[% END %]
[% result_count; result_count = result_count + 1 %].
diff --git a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2 b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
index 0a6d5a9371..95ba8ffc43 100644
--- a/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
+++ b/Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2
@@ -17,8 +17,8 @@
});
END;
%]
-[% IF NOT sms_carrier_hide_label; l('Mobile carrier:'); END; %]
-<select name="sms_carrier">
+[% IF NOT sms_carrier_hide_label; '<label for="sms_carrier">' _ l('Mobile carrier:') _ '</label>'; END; %]
+<select name="sms_carrier" id="sms_carrier" [% IF sms_carrier_hide_label; 'aria-label="' _ l('Mobile carrier') _ '"'; END; %]>
<option value="">[% l('Please select your mobile carrier') %]</option>
[% FOR carrier IN carriers.sort('name','region') -%]
<option value='[% carrier.id | html %]'[%
diff --git a/Open-ILS/src/templates/opac/parts/sms_number_textbox.tt2 b/Open-ILS/src/templates/opac/parts/sms_number_textbox.tt2
index e8d7eb8029..395b5a36d1 100644
--- a/Open-ILS/src/templates/opac/parts/sms_number_textbox.tt2
+++ b/Open-ILS/src/templates/opac/parts/sms_number_textbox.tt2
@@ -1,4 +1,5 @@
-[% IF NOT sms_number_hide_label; l('Mobile number:'); END; %]
+[% IF NOT sms_number_hide_label; '<label for="sms_notify">' _ l('Mobile number:') _ '</label>'; END; %]
<input type="text" name="sms_notify" [% setting = 'opac.default_sms_notify';
-IF ctx.user_setting_map.$setting; %] value='[% ctx.user_setting_map.$setting | html %]' [% END %]/>
+IF ctx.user_setting_map.$setting; %] value='[% ctx.user_setting_map.$setting | html %]' [% END %]
+[% IF sms_number_hide_label; 'aria-label="' _ l('Mobile number') _ '"'; END; %] id="sms_notify" />
[% IF NOT sms_number_hide_hint; l('Hint: use the full 10 digits of your phone #, no spaces, no dashes'); END; %]
-----------------------------------------------------------------------
Summary of changes:
Open-ILS/src/templates/opac/browse.tt2 | 1 +
Open-ILS/src/templates/opac/myopac/update_email.tt2 | 4 ++--
Open-ILS/src/templates/opac/myopac/update_password.tt2 | 6 +++---
Open-ILS/src/templates/opac/parts/advanced/global_row.tt2 | 2 +-
Open-ILS/src/templates/opac/parts/advanced/numeric.tt2 | 2 +-
Open-ILS/src/templates/opac/parts/advanced/search.tt2 | 4 ++--
Open-ILS/src/templates/opac/parts/anon_list.tt2 | 9 +++++----
Open-ILS/src/templates/opac/parts/cart.tt2 | 2 +-
Open-ILS/src/templates/opac/parts/login/form.tt2 | 2 +-
Open-ILS/src/templates/opac/parts/place_hold.tt2 | 7 +++++--
Open-ILS/src/templates/opac/parts/result/table.tt2 | 1 +
Open-ILS/src/templates/opac/parts/sms_carrier_selector.tt2 | 4 ++--
Open-ILS/src/templates/opac/parts/sms_number_textbox.tt2 | 5 +++--
13 files changed, 28 insertions(+), 21 deletions(-)
hooks/post-receive
--
Evergreen ILS
More information about the open-ils-commits
mailing list