Bug 19413 - Move the location of the Ok and cancel buttons to the 008 cataloguing builder
Summary: Move the location of the Ok and cancel buttons to the 008 cataloguing builder
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: master
Hardware: All All
: P5 - low trivial (vote)
Assignee: Alex Buckley
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks: 19970
  Show dependency treegraph
 
Reported: 2017-10-05 02:02 UTC by Alex Buckley
Modified: 2019-10-14 19:57 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 19413 - Moving the OK and Cancel inputs to the top of the 008 builder (2.56 KB, patch)
2017-10-05 02:20 UTC, Alex Buckley
Details | Diff | Splinter Review
Bug 19413 - Moving the OK and Cancel inputs to the top of the 008 builder (2.25 KB, patch)
2017-10-05 04:23 UTC, Dilan Johnpullé
Details | Diff | Splinter Review
Bug 19413 - Followup Changed fieldset id value (1.90 KB, patch)
2017-10-11 21:45 UTC, Alex Buckley
Details | Diff | Splinter Review
Bug 19413 - Followup Changed fieldset id value (2.09 KB, patch)
2017-10-12 00:05 UTC, Dilan Johnpullé
Details | Diff | Splinter Review
Bug 19413 - Moving the OK and Cancel inputs to the top of the 008 builder (2.35 KB, patch)
2017-10-13 08:55 UTC, Marcel de Rooy
Details | Diff | Splinter Review
Bug 19413 - Followup Changed fieldset id value (2.18 KB, patch)
2017-10-13 08:55 UTC, Marcel de Rooy
Details | Diff | Splinter Review
Screenshot of translated Plugin (183.25 KB, image/png)
2018-01-13 08:19 UTC, Katrin Fischer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Buckley 2017-10-05 02:02:28 UTC
The inputs that librarians are most likely to interact with on the Cataloguing 008 Builder are types of materials, and dates which are all at the top of the window.

The patch attached to this bug report will place the OK and cancel inputs at the top of the page therefore improving the UX of the page because they don't have to scroll past a whole lot of unnecessary inputs
Comment 1 Alex Buckley 2017-10-05 02:20:55 UTC Comment hidden (obsolete)
Comment 2 Dilan Johnpullé 2017-10-05 04:23:20 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2017-10-06 09:59:55 UTC
I see that git grep "id=\"action\"" returns various results.
Please explain why your change in staff-global would not have various side effects?
Comment 4 Alex Buckley 2017-10-11 21:45:59 UTC
Created attachment 67979 [details] [review]
Bug 19413 - Followup Changed fieldset id value

Sponsored-By: Catalyst IT
Comment 5 Alex Buckley 2017-10-11 21:47:27 UTC
Hi Marcel 

Apologies I made a mistake in not making the id value more specific. I have made the id value "fixedlengthbuilderaction" in the followup patch I have just attached to this bug report. Please test again
Comment 6 Dilan Johnpullé 2017-10-12 00:05:20 UTC Comment hidden (obsolete)
Comment 7 Marcel de Rooy 2017-10-13 08:55:46 UTC
Created attachment 68053 [details] [review]
Bug 19413 - Moving the OK and Cancel inputs to the top of the 008 builder

Followed test plan and patch applies and functions as described
Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Marcel de Rooy 2017-10-13 08:55:51 UTC
Created attachment 68054 [details] [review]
Bug 19413 - Followup Changed fieldset id value

Followed test plan again and patch still applies and works as expected.

Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 9 Marcel de Rooy 2017-10-13 08:59:42 UTC
(In reply to Alex Buckley from comment #5)
> Hi Marcel 
> 
> Apologies I made a mistake in not making the id value more specific. I have
> made the id value "fixedlengthbuilderaction" in the followup patch I have
> just attached to this bug report. Please test again

Thx. Cool name ;)

QA Comment:
As mentioned in first comment, this will help librarians only adjusting information at the top of the form. Sounds reasonable.
On the other hand those librarians that need to adjust something too at the end will now complain :) Would be interesting to make it floating like the cataloguing editor itself? No blocker for me. Especially since hitting Enter here submits the form too..
Comment 10 Jonathan Druart 2017-10-13 17:01:07 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 11 Katrin Fischer 2018-01-13 08:19:15 UTC
Created attachment 70480 [details]
Screenshot of translated Plugin

This doesn't work so well with languages where the term "Cancel" is translated to a longer word.