From: Bill Erickson <berick@esilibrary.com>
Date: Tue, 11 Feb 2014 14:16:09 +0000 (-0500)
Subject: LP#1268636 add expressive facet link titles
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=40a5441e8efe575fab1219b2702bf0b86be24999;p=evergreen%2Fpines.git

LP#1268636 add expressive facet link titles

Let the user know exactly what clicking on a facet will do.

Thanks to Dan Scott for the suggestion.

* Note that accessibility validators may dislike the repetitive titles,
  but clearly adding them provides more information (for everyone).
  Plus, as facets pile up, there's no reasonable way to differentiate
  between all titles, anyway, without title attributes becoming too
  long and unreadable.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
---

diff --git a/Open-ILS/src/templates/opac/parts/result/facets.tt2 b/Open-ILS/src/templates/opac/parts/result/facets.tt2
index 5f0e030fce..c4adc087a8 100644
--- a/Open-ILS/src/templates/opac/parts/result/facets.tt2
+++ b/Open-ILS/src/templates/opac/parts/result/facets.tt2
@@ -111,9 +111,13 @@ FOR facet IN sorted_facets;
                     <div class="facet_template facet_template_selected">
                         <div class="facet">
                             [% IF new_facets.size == 0 %]
-                            <a href="[% mkurl('', {}, ['facet']) %]">[% display_value %]</a>
+                            <a
+                              title="[% l('Show all results, not just those matching this term') %]"
+                              href="[% mkurl('', {}, ['facet']) %]">[% display_value %]</a>
                             [% ELSE %]
-                            <a href="[% mkurl('', {facet => new_facets}) %]">[% display_value %]</a>
+                            <a
+                              title="[% l('Expand to also show results not matching this term') %]"
+                              href="[% mkurl('', {facet => new_facets}) %]">[% display_value %]</a>
                             [% END %]
                         </div>
                         <div class="count">([% facet_data.count %])</div>
@@ -125,7 +129,11 @@ FOR facet IN sorted_facets;
                 %]
                     <div class="facet_template">
                         <div class="facet">
-                            <a href='[% mkurl('', {facet => selected_facets.merge([param_string])}, ['page']) %]'>[% display_value %]</a>
+                            <a
+                              title="[% l('Limit to results matching this term') %]"
+                              href='[% mkurl('', {facet => selected_facets.merge([param_string])}, ['page']) %]'>
+                              [% display_value %]
+                            </a>
                         </div>
                         <div class="count">([% facet_data.count %])</div>
                     </div>