Bug 15925

Summary: Correct some markup issues with patron lists pages
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: brendan, julian.maurice
Version: master   
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: 15916    
Bug Blocks:    
Attachments: Bug 15925 - Correct some markup issues with patron lists pages
Bug 15925 [Revised] Correct some markup issues with patron lists pages
Bug 15925 [Revised] Correct some markup issues with patron lists pages
Bug 15925 [Revised] Correct some markup issues with patron lists pages
[3.22.x] Bug 15925: Correct some markup issues with patron lists pages

Description Owen Leonard 2016-02-26 14:31:16 UTC
There are some markup issues with the patron lists templates which I would like to correct. I'm marking this bug dependent on Bug 15916 because the JS errors will make it hard to test properly.
Comment 1 Owen Leonard 2016-02-29 16:24:59 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2016-02-29 16:55:58 UTC Comment hidden (obsolete)
Comment 3 Frédéric Demians 2016-03-03 09:14:59 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2016-03-03 09:26:00 UTC
Created attachment 48609 [details] [review]
Bug 15925 [Revised] Correct some markup issues with patron lists pages

This patch makes some markup changes to the patron lists interface in
order to make them more consistent with other areas of Koha.

Change to look for when testing:

Tools -> Patron lists:

- Wrap "New patron list" button in standard <div id="toolbar">
- Place toolbar above page heading.

Tools -> Patron lists -> New patron list:

- Use built-in form validation for required fields
- Clarify page title and breadcrumbs on list add and edit

Validation errors have been corrected on all three templates, including
removal of an extra </div>.

Revision: I was wrong about the frequency of use of "Modify." There's no
clear-cut reason to replace it with "Edit" so I have undone that change.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
  Works as expected. No regression. Better texts. Consistent validation.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Brendan Gallagher 2016-03-03 22:31:43 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks!
Comment 6 Julian Maurice 2016-03-07 13:06:45 UTC
Created attachment 48746 [details] [review]
[3.22.x] Bug 15925: Correct some markup issues with patron lists pages

Patch for 3.22.x
Comment 7 Julian Maurice 2016-03-07 13:06:58 UTC
Patch pushed to 3.22.x, will be in 3.22.5