Based on features F10 and F11 developed for Lyon 2 University by Progilone.
After a search is performed, add in detail pages 3 links :
- Back to results : Leads to current search results page (same index, query and limits). Not shown if search leads directly to a detail page.
- Back to simple search : Leads to main page with header filled with current search index, query and limits (like result page).
- Back to advanced search : Leads to advanced search page.
Also adds 'focus' class to query input field in header.
Created attachment 3803 [details] [review]
Created attachment 3808 [details] [review]
Proposed patch (corrected)
Recreated patch, a new file was missing.
This has missed the feature freeze for 3.4.0 I'm afraid, but once it is signed off it can go into master as soon as the rel_3_4_0 branch is created
Created attachment 3842 [details]
Details page view
Screen shot of biblio details page
This patch doesn't apply anymore on HEAD. Please rebase it.
I object to the inclusion of a "back to simple search" link. Does this take you to the home page? There's a simple search form on every page. Aren't we encouraging the user to think that they must return to the home page to perform another quick search?
(In reply to comment #6)
> I object to the inclusion of a "back to simple search" link. Does this take you
> to the home page? There's a simple search form on every page. Aren't we
> encouraging the user to think that they must return to the home page to perform
> another quick search?
Not really, the link leads to home page but with search toolbar filled with previous search. So user can modify or relaunch it.
Bu the way,
I think your right, it whould be better to fill the current search in details page. So user doesn't need a link to see his current simple search.
I'll try work on it soon.
Created attachment 4150 [details] [review]
Proposed patch (updated)
Proposed patch updated for 3.4 (with TT)
Created attachment 4151 [details]
Details page view (updated)
Details view page.
I changed the patch and reduced behavior :
Only one link appears in actions list to come back to current search results.
Simple and advanced search can be relaunched in any page because they are in the header, no link needed.
Hope it is better.
Thanks for signing-off.
I think this updated patch is a great improvement. I did notice a couple of issues with it:
1. It doesn't remember the sorting options from the advanced search page.
If you go "back to results" it will default to a "relevance" sort.
2. If you do a search without a keyword you won't see a "back to results"
link on the detail page for any of your results. For instance, a
search where the user only checks an itemtype limit or a date limit.
3. If you were on multi-pages result page, and that you were on let say
page 5, if you click on Back to results, you are redirected on the
first page of the result page not on the 5th. The 'offset' parameter
should be saved in session and used to generated Back to results link.
So I should store in session the entire search URL ?
Created attachment 4276 [details] [review]
Proposed patch (full url)
Link leads to results page with all URL parameters : sort_by, ...
Created attachment 4285 [details] [review]
Thanks Fridolyn. Point 1, 2, 3 are solved with your last patch.
Created attachment 4925 [details] [review]
proposed patch (passed QA)
* works as announced
* nice feature, graphically consistent
* no comment on code itself
Marking passed QA
This patch overlap with bug 6483, that does much more. I remove the "passed QA" status, to avoid this patch being applied and causing problems to 6483.
Let's discuss of options we have (here or in koha-devel ?)
I think 6483 can be applied and this one discarded.
If it matters, I kind of like the look of the patch for bug 6483 better ...
Updating version : This ENH could be in Koha 3.8
*** This bug has been marked as a duplicate of bug 6483 ***