Owen Leonard
3b47f87b0f
This patch updates some markup and CSS so that the process of selecting a club from search results during the hold process looks the same as when selecting a patron: Club names should be links (like patron names are) and the table row should have a hover color to help indicate that it is clickable. This patch also changes the markup of the Patrons/Clubs tabs a little bit to make the information clearer: Superfluous Bootstrap-related tab markup has been removed, and the patron and club search results have been moved into their respective tab containers. This means that if you search for a patron but then switch to the clubs tab the patron list doesn't still display. To test, apply the patch and rebuild the staff client CSS (https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client) - You should have more than one patron club defined. - Begin the process of placing a hold on a title. - Test the process of searching for both patrons and clubs. - In each case the name (patron or club) should be an active link. - Hovering your mouse over the table rows should change the row background to yellow. - Clicking the other tab at this stage should hide the search results from your last search. - Whether you click the linked name or elsewhere in the table row you should be correctly redirected to the next step in the holds process. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
37 lines
1.9 KiB
HTML
37 lines
1.9 KiB
HTML
[% USE KohaDates %]
|
|
<table id="table_borrowers" class="selections-table">
|
|
<thead>
|
|
<tr>
|
|
<th>Name</th>
|
|
<th>Card number</th>
|
|
<th>Date of birth</th>
|
|
<th>Category</th>
|
|
<th>Library</th>
|
|
<th>Address</th>
|
|
<th>Primary phone</th>
|
|
</tr>
|
|
</thead>
|
|
<tbody>
|
|
[% FOREACH borrower IN borrowers %]
|
|
[% IF destination == "circ" %]
|
|
<tr class="clickable" data-url="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% borrower.borrowernumber | html %]">
|
|
<td><a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% borrower.borrowernumber | uri %]">[% borrower.surname | html %], [% borrower.firstname | html %]</a></td>
|
|
[% ELSIF destination == "holds" %]
|
|
[% IF multi_hold %]
|
|
[% SET data_url = "/cgi-bin/koha/reserve/request.pl?borrowernumber=" _ borrower.borrowernumber _ "&multi_hold=1&biblionumbers=" _ biblionumbers %]
|
|
[% ELSE %]
|
|
[% SET data_url = "/cgi-bin/koha/reserve/request.pl?borrowernumber=" _ borrower.borrowernumber _ "&biblionumber=" _ biblionumber %]
|
|
[% END %]
|
|
<tr class="clickable" data-url="[% data_url | url %]">
|
|
<td><a href="[% data_url | url %]">[% borrower.surname | html %], [% borrower.firstname | html %]</a></td>
|
|
[% END %]
|
|
<td>[% borrower.cardnumber | html %]</td>
|
|
<td>[% borrower.dateofbirth | $KohaDates %]</td>
|
|
<td>[% Categories.GetName( borrower.categorycode ) | html %]</td>
|
|
<td>[% Branches.GetName( borrower.branchcode ) | html %]</td>
|
|
<td>[% borrower.address | html %]</td>
|
|
<td>[% borrower.phone | html %]</td>
|
|
</tr>
|
|
[% END %]
|
|
</tbody>
|
|
</table>
|