Bug 25430: Improve the styling of the claims returned tab

It has been suggested that it would be useful to add title attributes to the different numbers on the return claims tab in the checkouts and patron details pages. In addition, color coding them ( and removing the slash ) will make the numbers easier to read.

Test Plan:
1) Set up return claims for your system
2) Apply this patch
3) View the claims tab for someone with not claims, and someone with claims
4) Note the resolved claims are green and have a "tooltip" when hovered over
5) Note the unresolved claims are green if 0, and yellow if not 0, and
   also have a "tooltip"

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Kyle M Hall 2020-05-08 13:15:40 -04:00 committed by Jonathan Druart
parent af0929abbb
commit 52f91af5bc
2 changed files with 8 additions and 12 deletions

View File

@ -873,17 +873,15 @@
<li>
[% IF ( patron.return_claims.count ) %]
<a href="#return-claims" id="return-claims-tab">
<span id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
/
<span id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
Claim(s)
<span title="Resolved claims" class="label label-success" id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
<span title="Unresolved claims" class="label label-warning" id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
</a>
[% ELSE %]
<a href="#return-claims" id="return-claims-tab">
<span id="return-claims-count-resolved">0</span>
/
<span id="return-claims-count-unresolved">0</span>
Claim(s)
<span title="Resolved claims" class="label label-success" id="return-claims-count-resolved">0</span>
<span title="Unresolved claims" class="label label-success" id="return-claims-count-unresolved">0</span>
</a>
[% END %]
</li>

View File

@ -763,17 +763,15 @@
<li>
[% IF ( patron.return_claims.count ) %]
<a href="#return-claims" id="return-claims-tab">
<span id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
/
<span id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
Claim(s)
<span title="Resolved claims" class="label label-success" id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
<span title="Unresolved claims" class="label label-warning" id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
</a>
[% ELSE %]
<a href="#return-claims" id="return-claims-tab">
<span id="return-claims-count-resolved">0</span>
/
<span id="return-claims-count-unresolved">0</span>
Claim(s)
<span title="Resolved claims" class="label label-success" id="return-claims-count-resolved">0</span>
<span title="Unresolved claims" class="label label-success" id="return-claims-count-unresolved">0</span>
</a>
[% END %]
</li>