Bug 18528

Summary: Patron card creator template: switch form fields for card height and card width
Product: Koha Reporter: Marc Véron <veron>
Component: Label/patron card printingAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: m.de.rooy
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18465
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
Bug 18528: Patron card creator template: switch form fields for card height and card width
Bug 18528: Patron card creator template: switch form fields for card height and card width
Bug 18528: Patron card creator template: switch form fields for card height and card width

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!