Bug 18528 - Patron card creator template: switch form fields for card height and card width
Summary: Patron card creator template: switch form fields for card height and card width
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Label/patron card printing (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Marc Véron
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-03 14:14 UTC by Marc Véron
Modified: 2018-06-04 20:18 UTC (History)
1 user (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
Confusing fields for height and width (20.57 KB, image/png)
2017-05-03 14:22 UTC, Marc Véron
Details
Bug 18528: Patron card creator template: switch form fields for card height and card width (2.12 KB, patch)
2017-05-03 14:31 UTC, Marc Véron
Details | Diff | Splinter Review
Bug 18528: Patron card creator template: switch form fields for card height and card width (2.22 KB, patch)
2017-05-08 22:48 UTC, Alex Buckley
Details | Diff | Splinter Review
Bug 18528: Patron card creator template: switch form fields for card height and card width (2.31 KB, patch)
2017-10-06 05:42 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Véron 2017-05-03 14:14:06 UTC
The form to create or edit a card template has a confusing sequence of fields for page and card height and width:
Page width - Page height - Card height - Card width

Switch Card height and Card width to make it more intuitive.
Comment 1 Marc Véron 2017-05-03 14:22:00 UTC
Created attachment 63014 [details]
Confusing fields for height and width
Comment 2 Marc Véron 2017-05-03 14:31:06 UTC
Created attachment 63017 [details] [review]
Bug 18528: Patron card creator template: switch form fields for card height and card width

This patch changes the sequnce of form fields for card height and card width.

To test:
- Apply patch
- Edit or create a patron card template
- Verify that the fields display in sequence Page height - Page width -Card height - Card width
Comment 3 Alex Buckley 2017-05-08 22:48:59 UTC
Created attachment 63262 [details] [review]
Bug 18528: Patron card creator template: switch form fields for card height and card width

This patch changes the sequnce of form fields for card height and card width.

To test:
- Apply patch
- Edit or create a patron card template
- Verify that the fields display in sequence Page height - Page width -Card height - Card width

Followed test plan, worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Comment 4 Marc Véron 2017-05-18 07:13:31 UTC Comment hidden (obsolete)
Comment 5 Marc Véron 2017-06-20 15:15:11 UTC
Still applies on current master
Comment 6 Marcel de Rooy 2017-10-06 05:42:46 UTC
Created attachment 67664 [details] [review]
Bug 18528: Patron card creator template: switch form fields for card height and card width

This patch changes the sequnce of form fields for card height and card width.

To test:
- Apply patch
- Edit or create a patron card template
- Verify that the fields display in sequence Page height - Page width -Card height - Card width

Followed test plan, worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Jonathan Druart 2017-10-09 19:17:52 UTC
Pushed to master for 17.11, thanks to everybody involved!