From: Bill Erickson <berick@esilibrary.com>
Date: Fri, 20 Dec 2013 16:42:06 +0000 (-0500)
Subject: LP#1268636 more expressive <a> titles in search results
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=3e1e2b918368d6b921b9aad2c45e0b0cc0878760;p=evergreen%2Fpines.git

LP#1268636 more expressive <a> titles in search results

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/table.tt2 b/Open-ILS/src/templates/opac/parts/result/table.tt2
index 06a7c69436..5faed1dc7c 100644
--- a/Open-ILS/src/templates/opac/parts/result/table.tt2
+++ b/Open-ILS/src/templates/opac/parts/result/table.tt2
@@ -25,7 +25,7 @@
     [% INCLUDE 'opac/parts/result/facets.tt2' %]
     </div>
             <div id="result_block" class="result_block_visible">
-                <table id="result_table_table"
+                <table id="result_table_table" title="[% l('Search Results') %]"
                   class="table_no_border_space table_no_cell_pad">
                     <tbody id="result_table">
                     [%  FOR rec IN ctx.records;
@@ -91,7 +91,7 @@ END;
 -%]
 
                                                     </div>
-                                                    <div id='bib_format' class='result_table_title_cell'>
+                                                    <div class='result_table_title_cell'>
                                                     [%- IF attrs.format_label; %]
                                                         <img title="[% attrs.format_label | html %]" alt="[% attrs.format_label | html %]" src="[% attrs.format_icon %]" /> [% attrs.format_label; %]
                                                     [%- END %]
@@ -110,7 +110,9 @@ END;
                                                             class='classic_link hide_me'
                                                             name="googleBooks-link">[% l("Browse in Google Books Search") %]</a>
                                                     </span>
-                                                    <table class="table_no_border_space table_no_cell_pad table_no_border results_info_table">
+                                                    <table 
+                                                       title="[% l('Record Holdings Summary') %]"
+                                                       class="table_no_border_space table_no_cell_pad table_no_border results_info_table">
                                                         [% IF args.holdings.size > 0 %]
                                                         <tr name='bib_cn_list' class='result_table_title_cell'>
                                                             <td valign='top'>
@@ -232,7 +234,8 @@ END;
                                                             %]
                                                             <tr name='bib_cn_list' class='result_table_title_cell'>
                                                                 <td colspan='2'>
-                                                                    <table class='result_holdings_table'>
+                                                                    <table title="[% l('Record Holdings Details') %]"
+                                                                           class='result_holdings_table'>
                                                                         <thead><tr>
                                                                             <th>[% l('Library') %]</th>
                                                                             <th>[% l('Shelving location') %]</th>
@@ -294,6 +297,7 @@ END;
                                                         <div class="results_aux_utils place_hold"><a
                                                                 href="[% mkurl(ctx.opac_root _ '/place_hold',
                                                                     {hold_target => rec.id, hold_type => 'T', hold_source_page => mkurl()}, ['query']) %]"
+                                                                [% HTML.attributes(title => l('Place Hold on [_1]', attrs.title)) %]
                                                                     name="place_hold_link" class="no-dec"><img
                                                                 src="[% ctx.media_prefix %]/images/green_check.png"
                                                                 alt=""/><span class="result_place_hold">[% l('Place Hold') %]</span></a>
@@ -305,11 +309,14 @@ END;
                                                             %]
                                                             [%  ELSE;
                                                                 operation = ctx.mylist.grep(rec.id).size ? "delete" : "add";
-                                                                label = (operation == "add") ? l("Add to my list") : l("Remove from my list");
+                                                                label = (operation == "add") ?  l("Add to my list") : l("Remove from my list");
+                                                                title_label = (operation == "add") ? 
+                                                                  l("Add [_1] to my list", attrs.title) : 
+                                                                  l("Remove [_1] from my list", attrs.title);
                                                                 href = mkurl(ctx.opac_root _ '/mylist/' _ operation, 
                                                                         {record => rec.id, anchor => 'record_' _ rec.id}, 1);
                                                             %]      
-                                                            <a href="[% href %]" class="no-dec">
+                                                            <a href="[% href %]" class="no-dec" [% HTML.attributes(title => title_label) %]>
                                                                 <img src="[% ctx.media_prefix %]/images/clipboard.png" alt="" />
                                                                 [% label %]
                                                             </a>
@@ -318,11 +325,12 @@ END;
                                                         [% IF ENV.OILS_CONTENT_CAFE_USER %]
                                                         [% ident = attrs.isbn_clean || attrs.upc %]
                                                         <div class="results_aux_utils result_util">
-                                                            <a title="[% l('Reviews and More') %]" target='_blank' 
+                                                            <a target='_blank' 
+                                                               [% HTML.attributes(title => l('Reviews and More for [_1]', attrs.title)) %]
                                                                 href="[% ctx.ext_proto %]://contentcafe2.btol.com/ContentCafeClient/ContentCafe.aspx?UserID=[%- 
                                                                     ENV.OILS_CONTENT_CAFE_USER %]&amp;Password=[%-
                                                                     ENV.OILS_CONTENT_CAFE_PASS %]&amp;ItemKey=[% ident | uri %]&amp;Options=Y">
-                                                                <img src='[% ctx.media_prefix %]/images/starz.png'/> 
+                                                                <img src='[% ctx.media_prefix %]/images/starz.png' alt="[% l('Ratings Stars') %]"/> 
                                                                 <span class="results_reviews">[% l('Reviews &amp; More') %]</span>
                                                             </a>
                                                         </div>