Bug 19970

Summary: Revise change of bug 19413 to work better for translations
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: CatalogingAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, m.de.rooy, martin.renvoize, nick, oleonard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 19413    
Bug Blocks:    
Attachments: screenshot de-DE
Bug 19970: Revise change of bug 19413 to work better for translations
Bug 19970: Revise change of bug 19413 to work better for translations
Bug 19970: Revise change of bug 19413 to work better for translations
Bug 19970: (17.05.x) Revise change of bug 19413 to work better for translations
Bug 19970: (17.11.x) Revise change of bug 19413 to work better for translations

Description Katrin Fischer 2018-01-13 08:22:03 UTC
Created attachment 70481 [details]
screenshot de-DE

Bug 19413 moved the buttons in the cataloguing plugins to the top, this doesn't work well for translations where the terms 'cancel' and 'ok' can be longer.
Comment 1 Owen Leonard 2018-05-31 15:45:24 UTC Comment hidden (obsolete)
Comment 2 Maryse Simard 2018-06-08 19:45:59 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2018-06-14 19:54:55 UTC
Created attachment 76070 [details] [review]
Bug 19970: Revise change of bug 19413 to work better for translations

This patch changes how the submit and cancel buttons appear in the MARC
008 plugin window. The controls are now displayed as buttons in a
floating toolbar.

To test you must have MARC tag 008 configured to use the
marc21_field_008.pl plugin.

- Apply the patch and open an existing or blank record in cataloging.
- Click the plugin link next to tag 008 to trigger the popup window.
- Confirm that "Save" and "Cancel" appear correctly in a toolbar.
- Scroll down the page to confirm that the toolbar "sticks" to the top
  of the window.
- Confirm that the "Save" and "Cancel" buttons work correctly.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Followed the test plan and the patch works.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2018-06-14 19:55:50 UTC
I guess it would be useful to have the result floating as well (?)
Comment 5 Nick Clemens 2018-06-15 10:44:05 UTC
Awesome work all!

Pushed to master for 18.11
Comment 6 Martin Renvoize 2018-06-15 12:12:00 UTC
Pushed to 18.05.x for 18.05.01
Comment 7 Fridolin Somers 2018-06-18 06:58:31 UTC
Hi, I'm not sure how to apply on 17.11.x since it does not have Asset TT plugin (Bug 20538).
Can you provide a rebased patch ?
Comment 8 Owen Leonard 2018-06-18 11:59:23 UTC Comment hidden (obsolete)
Comment 9 Owen Leonard 2018-06-18 12:06:41 UTC
Created attachment 76119 [details] [review]
Bug 19970: (17.11.x) Revise change of bug 19413 to work better for translations

This patch changes how the submit and cancel buttons appear in the MARC
008 plugin window. The controls are now displayed as buttons in a
floating toolbar.

To test you must have MARC tag 008 configured to use the
marc21_field_008.pl plugin.

- Apply the patch and open an existing or blank record in cataloging.
- Click the plugin link next to tag 008 to trigger the popup window.
- Confirm that "Save" and "Cancel" appear correctly in a toolbar.
- Scroll down the page to confirm that the toolbar "sticks" to the top
  of the window.
- Confirm that the "Save" and "Cancel" buttons work correctly.
Comment 10 Fridolin Somers 2018-06-18 12:13:01 UTC
Pushed to 17.11.x for 17.11.07