https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33190

            Bug ID: 33190
           Summary: Add search history button to advance search form if
                    EnableSearchHistory keep
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: Searching
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]

Created attachment 148047
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148047&action=edit
screen shot of search history button in koha dev box as preview

Hi All,

I believe the 'search history' feature to be quite useful in daily repeated
search work flows, further to this it is a feature fairly hidden behind a drop
down UI in the top right when logged in. 

Therefor to promote the use of this valuable tool and also to improve it's
discoverability, I propose the follow new enhancement.

User story:
* Add 'search history' button to advanced search button groups if
'EnableSearchHistory' sys pref is set to 'keep'

Key logic:
* If 'EnableSearchHistory' sys pref is set to 'keep' (this is an existing sys
pref), an additional 'search history' button will populate at the of the
current advance search buttons group
* This new button element makes use of the existing syntax for other button on
page and loads the font awesome 4.7 history icon which aligns with the buttons
action.
* The button will not display unless 'EnableSearchHistory' sys pref is set to
'keep', which makes use of a IF statement in the advsearch.tt template file at
around line 100 ish (see patch)

```
[% IF Koha.Preference('EnableSearchHistory') %]
<div class="btn-group">
<a href="/cgi-bin/koha/catalogue/search-history.pl" class="btn btn-link"><i
class="fa fa-history"></i> search history</a>
</div>
[% END %]

```

I have tested in Koha dev box v22 and all works well. I will create test plan
in patch commit message.

I am creating my first patch for this very soon, just reading coding guidelines
to ensure I patch as community expects.

See attached screen shot.

I welcome feedback.

Thanks.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to