Bug 8664

Summary: Add floating toolbar with save button to system preferences editor
Product: Koha Reporter: Owen Leonard <oleonard>
Component: System AdministrationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: enhancement    
Priority: P5 - low CC: chris, gmcharlt, jcamins, jonathan.druart, martin.renvoize, mirko, nengard
Version: 3.12   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3988
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 8664 - Add floating toolbar with save button to system preferences editor
Bug 8664 - Add floating toolbar with save button to system preferences editor

Description Owen Leonard 2012-08-21 20:29:21 UTC
I propose duplicating the system preferences "save" button in a toolbar at the top of the system preferences page which will "stick" to the top of the page as you scroll down, just as the save button on addbiblio.pl does. This would make it easy to save your changes without scrolling all the way to the bottom of the screen.

The toolbar would not appear when the page displays search results since in that situation multiple save buttons might appear on the page.
Comment 1 Owen Leonard 2012-10-25 18:14:20 UTC Comment hidden (obsolete)
Comment 2 Mirko Tietgen 2012-10-25 18:29:52 UTC
Created attachment 13060 [details] [review]
Bug 8664 - Add floating toolbar with save button to system preferences editor

This patch adds a copy of the "save" button in a toolbar at
the top of the system preferences editor. The toolbar becomes
fixed at the top of the page when you scroll down, exactly like
the toolbar on addbiblio.pl.

Because multiple save buttons might appear when the system prefs
page displays search results, the toolbar does not appear in
this case.

To test, open the system preferences editor and change a setting.
Try saving both with the button in the toolbar and the button at
the top of the page. Test scrolling to confirm that the toolbar
"sticks" to the top of the page when you scroll down.

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Comment 3 Martin Renvoize 2012-11-01 20:06:17 UTC
*** Bug 3988 has been marked as a duplicate of this bug. ***
Comment 4 Jonathan Druart 2012-11-19 13:48:02 UTC
QA Comments:

Works as expected.

Just one remark about indentation, I am not a big fan of this kind of indendation:
+    [% UNLESS ( searchfield ) %]$(document).ready(function(){
+            $('#toolbar').fixFloat();
+        });[% END %]

It is not blocker and we don't have enough guidelines on the template files structure.

So marked as Passed QA.
Comment 5 Jared Camins-Esakov 2012-11-23 17:20:59 UTC
This patch has been pushed to master.