From 9bc113b872193c8eea2d21c8525d041fb13c5f81 Mon Sep 17 00:00:00 2001
From: jpkanter <kanter@ub.uni-leipzig.de>
Date: Mon, 27 Mar 2023 09:52:23 +0200
Subject: [PATCH] refs #23451 [finc] accessibility changes to
advanced_search/layout
* changed language token on remove field to a more unique text that describes the action
* extended the clear button javascript to also clear number fields and reset the sliders (apparently not even real vufind got this)
* added a contengency function for clearing of the sliders in advanced search publishDate widget
---
local/languages/de.ini | 1 +
local/languages/en.ini | 1 +
themes/finc/js/advanced_search.js | 12 ++++++++++++
themes/finc/templates/search/advanced/layout.phtml | 3 +--
4 files changed, 15 insertions(+), 2 deletions(-)
diff --git a/local/languages/de.ini b/local/languages/de.ini
index 8dcdc1821bd..921a47e7822 100644
--- a/local/languages/de.ini
+++ b/local/languages/de.ini
@@ -484,6 +484,7 @@ Related Title = "Zugehörige Werke"
Renew Login = "Bitte, erneut einloggen"
Remove all Filters = "entfernen Sie alle Suchfilter"
Remove from Book Bag = "Nicht mehr merken"
+remove_adv_search_field = "Eingaben in diesem Suchfeld entfernen"
Repeat your password = "Wiederhole das Passwort"
Reserve = Vormerkbar
Reserved = "Vorgemerkte Medien"
diff --git a/local/languages/en.ini b/local/languages/en.ini
index 0f683fb5e77..c0b7ed4dc06 100644
--- a/local/languages/en.ini
+++ b/local/languages/en.ini
@@ -528,6 +528,7 @@ Recall This = "Recall this"
Recent acquisitions = "Recent Acquisitions"
Region = Region
Remove all Filters = "remove all Filters"
+remove_adv_search_field = "Remove input in this search field"
Reserves = Reserves
Results per page = "Results per Page"
Reviews = Reviews
diff --git a/themes/finc/js/advanced_search.js b/themes/finc/js/advanced_search.js
index 04598bd2a6a..f140183d2d9 100644
--- a/themes/finc/js/advanced_search.js
+++ b/themes/finc/js/advanced_search.js
@@ -160,10 +160,22 @@ deleteGroup = function _deleteGroup(group) {
$(document).ready(function advSearchReady() {
$('.clear-btn').click(function clearBtnClick() {
$('input[type="text"]').val('');
+ $('input[type="number"]').val('');
$('select').not('#limit').find('option:selected').prop("selected", false);
if (typeof defaultLimit !== 'undefined') {
$("#limit").val(defaultLimit);
}
+ // Resetting Bootstrap Sliders
+ let fromValue = Number($('#publishDatefrom').val());
+ let toValue = Number($('#publishDateto').val());
+ $('#publishDateSortdateSlider').slider(
+ 'setValue',
+ [
+ isNaN(fromValue) || fromValue <= 0 ? 1400 : fromValue,
+ isNaN(toValue) || toValue <= 0 ? (new Date().getFullYear() + 1) : toValue
+ ],
+ true
+ );
$("#illustrated_-1").click();
});
});
diff --git a/themes/finc/templates/search/advanced/layout.phtml b/themes/finc/templates/search/advanced/layout.phtml
index d3c451aa5cf..d2e5d1be249 100644
--- a/themes/finc/templates/search/advanced/layout.phtml
+++ b/themes/finc/templates/search/advanced/layout.phtml
@@ -137,8 +137,7 @@
</select>
</div>
<a href="javascript:void(0);" class="adv-term-remove hidden">
- <?php /* finc: add span for screenreader */ ?>
- <span class="sr-only"><?=$this->transEsc("remove")?></span>
+ <span class="sr-only"><?=$this->transEsc("remove_adv_search_field")?></span>
<i class="fa fa-times" aria-hidden="true"></i>
</a>
</fieldset>
--
GitLab