Bug 23448

Summary: Clean up subscription detail template
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, maryse.simard
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:
19.11.00
Circulation function:
Attachments: Bug 23448: Clean up subscription detail template
Bug 23448: (follow-up) Add comments on markup structure
Bug 23448: Clean up subscription detail template
Bug 23448: (follow-up) Add comments on markup structure
Bug 23448: Clean up subscription detail template
Bug 23448: (follow-up) Add comments on markup structure

Description Katrin Fischer 2019-08-10 20:11:58 UTC
The HTML validator complains about the placement of the </main> tag and a missing closing </div> element.
Comment 1 Owen Leonard 2019-08-12 18:12:59 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2019-08-12 18:13:01 UTC Comment hidden (obsolete)
Comment 3 Maryse Simard 2019-08-16 20:41:12 UTC Comment hidden (obsolete)
Comment 4 Maryse Simard 2019-08-16 20:41:14 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2019-08-17 19:12:29 UTC
Created attachment 92335 [details] [review]
Bug 23448: Clean up subscription detail template

This patch makes a number of corrections to the subscription detail
page: Re-indented consistently with four spaces. Removed invalid
attributes. Added missing </div>.

To test, apply the patch and go to Serials. Search for a subscription
and view the details page.

The page should look correct. Validating the source should return no
errors.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Katrin Fischer 2019-08-17 19:12:32 UTC
Created attachment 92336 [details] [review]
Bug 23448: (follow-up) Add comments on markup structure

This patch adds comments to the markup in order to highlight the
structure of the page.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Martin Renvoize (ashimema) 2019-08-19 09:25:21 UTC
Nice work!

Pushed to master for 19.11.00
Comment 8 Fridolin Somers 2019-09-05 15:02:42 UTC
Enhancement not pushed to 19.05.x