From: Kathy Lussier Date: Thu, 2 Mar 2017 04:06:49 +0000 (-0500) Subject: LP#1670425: Moving display of advanced search limiters on search results page X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=6fe64e67931f0090e50e5b5c4c88d2e2fe6acb1c;p=evergreen%2Fpines.git LP#1670425: Moving display of advanced search limiters on search results page Moves the new 2.12 advanced search filters block out of the sidebar and along the top of the screen so that the user can see limiters along with their other search terms. Signed-off-by: Kathy Lussier Signed-off-by: Ben Shum --- diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2 index 6d5ef52c6a..b8e650d2df 100644 --- a/Open-ILS/src/templates/opac/css/style.css.tt2 +++ b/Open-ILS/src/templates/opac/css/style.css.tt2 @@ -1054,6 +1054,42 @@ div.adv_search_available { margin: auto; } +/* styling for advanced search filters that display with searchbar */ + +#adv_filter_results_block h4 { display: inline; } + +#adv_filter_results_block { + margin-top: 10px; + margin-bottom: 16px; +} + +.adv_filter_results_block_label { + font-weight:bold; + padding: 4px 6px 4px 12px; +} + +.adv_filter_results_group { + font-size: 12px; + display: inline-block; + background: [% css_colors.accent_lighter2 %]; + border-style: solid; + border-color: [% css_colors.accent_medium %]; + border-width: 1px; + border-radius: 4px; + padding: 2px 4px 2px 4px; + margin-right:6px; +} + +.remove_filter { + font-size: 16px; + margin-left: 3px; + } + +a.remove_filter { + text-decoration: none; + color: [% css_colors.accent_dark %]; +} + [%- IF we_want_to_turn_on_facet_styling.defined; %] /* some facet styling */ .facetClassContainer { margin: 2px; border: 1px solid [% css_colors.accent_light %]; } diff --git a/Open-ILS/src/templates/opac/parts/result/adv_filter.tt2 b/Open-ILS/src/templates/opac/parts/result/adv_filter.tt2 index 07efb48498..d2c4e73343 100644 --- a/Open-ILS/src/templates/opac/parts/result/adv_filter.tt2 +++ b/Open-ILS/src/templates/opac/parts/result/adv_filter.tt2 @@ -11,15 +11,8 @@ FOR filter IN ctx.query_struct.filters; IF crad AND NOT pubdate_filters.grep('^' _ filter.name _ '$').size; remove_filter = 'fi:' _ fname; -%] -
-
- ✘ +

[% IF filter.negate; l('Not'); END %] [% (crad.description || crad.label) | html %]

-
-
-
[% temp = []; FOR fval IN fvalues; thing = ctx.search_ccvm('ctype',fname,'code',fval).0; @@ -30,63 +23,57 @@ FOR filter IN ctx.query_struct.filters; END; FOR display_value IN temp.sort; %] -
-
- [% display_value | html%] -
-
+ + [% display_value | html %] + [% UNLESS loop.last; + l('OR'); + END %] + + [% END; # FOR %] -
-
-
+ × +
[%- END; # IF crad -%] [%- IF filter.name == 'locations'; locs = ctx.search_acpl('id',filter.args) -%] -
-
- ✘ +

[% IF filter.negate; l('Not'); END %] [% l('Locations') %]

-
-
-
[% temp = []; FOR loc IN locs; temp.push(loc.name); END; FOR display_name IN temp.sort; %] -
-
+ [% display_name | html%] -
-
+ [% END; # FOR %] -
-
-
+ × + +
[%- END; # IF locations -%] [%- IF pubdate_filters.grep('^' _ filter.name _ '$').size; date1 = CGI.param('date1'); date2 = CGI.param('date2'); -%] -
-
- ✘ +

[% IF filter.negate; l('Not'); END %] [% l('Publication Year') %]

-
-
-
+ [% IF filter.name == 'date1' %][% l('[_1]', date1) %] [% ELSIF filter.name == 'before' %][% l('Before [_1]', date1) %] [% ELSIF filter.name == 'after' %][% l('After [_1]', date1) %] [% ELSIF filter.name == 'between' %][% l('Between [_1] and [_2]', date1, date2) %] [% END %] -
-
-
+ + × +
+ [%- END; # IF pubdate_filters -%] [%- END; # FOR -%] diff --git a/Open-ILS/src/templates/opac/parts/result/lowhits.tt2 b/Open-ILS/src/templates/opac/parts/result/lowhits.tt2 index cbaf0f2a22..cd6b5a935c 100644 --- a/Open-ILS/src/templates/opac/parts/result/lowhits.tt2 +++ b/Open-ILS/src/templates/opac/parts/result/lowhits.tt2 @@ -2,7 +2,6 @@

[% l('Search Results filters') %]

- [% INCLUDE 'opac/parts/result/adv_filter.tt2' %]
[% INCLUDE "opac/parts/staff_saved_searches.tt2" %] diff --git a/Open-ILS/src/templates/opac/parts/result/table.tt2 b/Open-ILS/src/templates/opac/parts/result/table.tt2 index 989f7e001f..a5034845ef 100644 --- a/Open-ILS/src/templates/opac/parts/result/table.tt2 +++ b/Open-ILS/src/templates/opac/parts/result/table.tt2 @@ -40,16 +40,6 @@

[% l('Saved Searches') %]

[% INCLUDE "opac/parts/staff_saved_searches.tt2" %] [%- END %] - [% IF ctx.query_struct.filters.size > 0 %] - [% stuff = INCLUDE 'opac/parts/result/adv_filter.tt2' %] - [% IF stuff %] -

[% l('Search Results filters') %]

-
-
[% l('Filtered By') %]
- [% stuff %] -
- [% END %] - [% END %]

[% l('Search Results facets') %]

[% INCLUDE 'opac/parts/result/facets.tt2' %]

[% l('Search Results List') %]

diff --git a/Open-ILS/src/templates/opac/parts/searchbar.tt2 b/Open-ILS/src/templates/opac/parts/searchbar.tt2 index f807134dca..757e3fee89 100644 --- a/Open-ILS/src/templates/opac/parts/searchbar.tt2 +++ b/Open-ILS/src/templates/opac/parts/searchbar.tt2 @@ -131,7 +131,17 @@ END; [% END %] [% END %] - [% IF (is_advanced AND NOT is_special) AND CGI.param('qtype') %] + [% IF ctx.query_struct.filters.size > 0 %] + [% stuff = INCLUDE 'opac/parts/result/adv_filter.tt2' %] + [% IF stuff %] +

[% l('Search Results filters') %]

+
+ [% l('Filtered by:') %] + [% stuff %] +
+ [% END %] + [% END %] + [% IF (is_advanced AND NOT is_special) AND CGI.param('qtype') %]
[% IF fcount > 0 %] [ [% l('[quant,_1,filter,filters] applied', fcount) %] ] @@ -141,6 +151,7 @@ END; %] ]
[% END %] +