From 59266b5965724051cee536e66ce71c6db18d1900 Mon Sep 17 00:00:00 2001
From: Mark Cooper <markchristophercooper@gmail.com>
Date: Wed, 5 Dec 2012 15:18:54 -0800
Subject: [PATCH] Add an 'X' icon to the left of facets to emphasize that
 selected facets can be removed by re-clicking the facet link. Icon is resized
 version of cancel-icon-u.png.

Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>
---
 Open-ILS/src/templates/opac/css/style.css.tt2       |   9 +++++++--
 Open-ILS/src/templates/opac/parts/result/facets.tt2 |   4 ++--
 Open-ILS/web/opac/images/facet_remove.png           | Bin 0 -> 1518 bytes
 3 files changed, 9 insertions(+), 4 deletions(-)
 create mode 100644 Open-ILS/web/opac/images/facet_remove.png

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index d97989dd3c..4ca248961d 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1000,7 +1000,7 @@ div.facet_sidebar {
 
 .facet_template .facet {
     float:left;
-    width:124px;
+    width:118px;
 }
 
 .facet_template .count {
@@ -1009,7 +1009,7 @@ div.facet_sidebar {
     padding-right:11px;
 }
 
-.facet_template_selected {
+.facet_template_selected .facet {
     background-color: [% css_colors.accent_lighter2 %];
 }
 
@@ -1018,6 +1018,11 @@ div.facet_sidebar {
     padding-right: 0.5em;
 }
 
+.facet_remove {
+    margin-left: -0.9em;
+    margin-right: 0.1em;
+}
+
 #footer-wrap {
     background: linear-gradient([% css_colors.primary_fade %], [% css_colors.primary %]);
     background: -moz-linear-gradient([% css_colors.primary_fade %], [% css_colors.primary %]);
diff --git a/Open-ILS/src/templates/opac/parts/result/facets.tt2 b/Open-ILS/src/templates/opac/parts/result/facets.tt2
index fed133ab8d..ec7a5cc789 100644
--- a/Open-ILS/src/templates/opac/parts/result/facets.tt2
+++ b/Open-ILS/src/templates/opac/parts/result/facets.tt2
@@ -103,9 +103,9 @@ 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 href="[% mkurl('', {}, ['facet']) %]"><img class="facet_remove" src="[% ctx.media_prefix %]/opac/images/facet_remove.png" />[% display_value %]</a>
                             [% ELSE %]
-                            <a href="[% mkurl('', {facet => new_facets}) %]">[% display_value %]</a>
+                            <a href="[% mkurl('', {facet => new_facets}) %]"><img class="facet_remove" src="[% ctx.media_prefix %]/opac/images/facet_remove.png" />[% display_value %]</a>
                             [% END %]
                         </div>
                         <div class="count">([% facet_data.count %])</div>
diff --git a/Open-ILS/web/opac/images/facet_remove.png b/Open-ILS/web/opac/images/facet_remove.png
new file mode 100644
index 0000000000000000000000000000000000000000..93949292505923a759b5569680516d1a5c32ed12
GIT binary patch
literal 1518
zcmV<K1rhp*P)<h;3K|Lk000e1NJLTq000UA000UI1^@s6jWW-@00004XF*Lt006JZ
zHwB96000CeX+uL$Nkc;*P;zf(X>4Tx05}naRo`#hR1`jmZ&IWdKOk5~hl<6oRa0BJ
z8yc;~21%2p?MfD<>DVeH<T^KrsT&8|>9(p*dx19w`~g7O0}n_%Aq@s%d)fBDv`JHk
zDym6Hd+5XuAtvnwRpGmKVkc9?T=n|PIo~<wJLg{8L_J?=wVD}Kh?c9aozEndlcyGx
zo=u9<v(!ri)T`-EEs@L35-!0N_s;9#9f}Cc?UC;OPWB_edW+oAi6T$HZWSGU8TbrQ
z%+zbPOBBBc`}k?M2Hf);@Y6N~0;>X-eVh__(Z?q}P9Z-Dj?gOW6|D%o20XmjW-qs4
zUjrD(li^iv8@eK9k+ZFmVRFymFOPAzG5-%Pn|1W;U4vNroTa&AxDScmEA~{ri9gr1
z^c?U@uwSpaNnw8l_>cP1d;)kMQS_;jeRSUEM_*s96y65j1$)tOrwdK{YIQMt92l|D
z^(E_=$Rjw{b!QT@q!)niR`|5oW9X5n$Wv+HVc@|^eX5yXnsHX<gx$-tTA9oOBadXi
zr_JPm2Y^4ct-PoO&C)tIGolvqOIK@duBk!Vu9{g<3;i;gJ6?~-DQ&xz!jvD&4!U-s
z8Os(*#?k2}f30SEXA#=i1-qUX+K`{!((H5w7<t$~ygD!D1{~X6)KX%$qrgY#L_{M_
z7A<1csY*MfP@XcB#Jxr~JS8&7goVS)VKE|4(h_Xlc{z{c$ApZs7riZ_QKdV_uW-M~
zu~<J-*#Z0?VzcZp8)p-wus7J7><CN2I>8PF3UX~a6)MwxDE0HaPjyrlI!;jX{6Kvu
zh*8ej?;85ekN$?5uuCiSBTvvVG+XTxAO{m@bvM#Jr)z6J><&E22D|vq?Y?Vkbo_Di
zjopiF$2PET#<s%v*srlI{B2SKJ79W>mZ8e<cESmGBON_l0n;T7>u=y$(ArYkv7@Ex
z`GL?QCc!_*KFrd&;n1r7qW-CFs9&fT)ZaU5gc&=gBz-D<EBz>aCw(vdOp0__x+47~
zU6sC(E(JNe@4cTT*n6*EVH4eoU1-&7pEV~_PRe`a7v+@vy!^5}8?Y3)UmlaE<h}6h
z3HHql{T;m+bPBU-O|^S1@dOw&4<!bj2G_<^#e}PL7FpY$lcrKO$i~?8Bd2y;oaL5^
zcsibnCrF9!i%-PI;xhubp1k;8_$IKX1NHus6EHeD;B72SCCD@4ojP$=Mf3`Eo6yZ&
zeg@wTqDiZE);7u&SJ|(suPF(9%D6IJ)klXF%`_Fy<tR3HxV^%Qqa?nAB97=m-uu2q
zcHInZ?ps8M|H3=#R%lzO6MgLv^}ib0hVV{&<};#;2lcwW;^(7C<OY#bI<VjS9qCKr
z-E_Cnc!2j+&nHAXA2%BRt~VMxUn2h&(Pi^LSpac(Y#S>R00009a7bBm000XU000XU
z0RWnu7ytkPU`a$lR2UgekULMpP#A{4mY%i(a*YCs;B8SCCr3;)G2mcORwwIt@ZXfh
zq@DZ${1Fyn9CQ$&G|_;Rq$uq<r$>s>@FwrKyidNHKyPp@BBK^@rzU#1zl9}D&uT_2
zo6&p-F2zVhY%viCT8(R7{aXLy+|vs^mPu<=K}4QRThh#NOf`|v0#yMenZj=MiU;2w
zBy#JR$p}RI9oc=`6gcPLp4atwI)mwo>Px_7pbRMSed%K;TW^Er#!I`g0K;*ka#hc6
zZ{fgkJRt}`P-Mkwcl3>J=Pt~}u!M?00Kgasl+H~6=CCCF$G@O6@0QABG4*G@fRF+b
zQV1b_KDXAe3j2!#1?RY8B$FwPAqm9c2zsymCj0WBSSn!1ra=6hfYr54DJiSkj4^`7
z4(j#o=4S7)5w7yAmY}#>z$_Lexu~kaxLm36qZ8iSIW)o#Ec*%ahMU{tgW^H$2R727
UvY5dP5dZ)H07*qoM6N<$g0mOYJOBUy

literal 0
HcmV?d00001

-- 
2.11.0