Bug 26093

Summary: Markup error after Bug 24279 creates formatting problem
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: RESOLVED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: andrew, bwsdonna, gmcharlt, jonathan.druart+koha, katrin.fischer, kelly, kyle.m.hall, kyle, lucas, m.de.rooy, martin.renvoize
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Text to go in the release notes:
Version(s) released in:
20.11.00
Bug Depends on: 24279    
Bug Blocks:    
Attachments: Bug 26093: Markup error after Bug 24279 creates formatting problem
Bug 26093: Markup error after Bug 24279 creates formatting problem
Bug 26093: Markup error after Bug 24279 creates formatting problem
Bug 26093: Markup error after Bug 24279 creates formatting problem

Description Owen Leonard 2020-07-29 17:16:58 UTC
After Bug 24279 the "Statuses" section on the item detail page (moredetail.pl) isn't formatted correctly. The ordered list no longer has the correct CSS applied because the container changed from <div> to <span>
Comment 1 Owen Leonard 2020-07-29 20:12:55 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2020-07-30 11:38:27 UTC
Created attachment 107575 [details] [review]
Bug 26093: Markup error after Bug 24279 creates formatting problem

This patch makes a few minor changes to the markup of the item detail
page in order for everything to be valid and styled correctly:

- Remove unnecessary and invalid <br clear="both"/>
- Move heading inside container so that floats clear correctly
- Change <span> tag back to <div> to enable the correct style.

To test, apply the patch and view the item details page for a title with
multiple items.

 - Each section starting with "Barcode" should be bordered above by a
   two-pixel gray line
 - The "statuses" section should be styled like the areas above and
   below, not as an ordered list.
Comment 3 Andrew Fuerste-Henry 2020-07-30 11:46:54 UTC
Created attachment 107578 [details] [review]
Bug 26093: Markup error after Bug 24279 creates formatting problem

This patch makes a few minor changes to the markup of the item detail
page in order for everything to be valid and styled correctly:

- Remove unnecessary and invalid <br clear="both"/>
- Move heading inside container so that floats clear correctly
- Change <span> tag back to <div> to enable the correct style.

To test, apply the patch and view the item details page for a title with
multiple items.

 - Each section starting with "Barcode" should be bordered above by a
   two-pixel gray line
 - The "statuses" section should be styled like the areas above and
   below, not as an ordered list.

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Comment 4 Katrin Fischer 2020-07-30 20:50:47 UTC
Created attachment 107610 [details] [review]
Bug 26093: Markup error after Bug 24279 creates formatting problem

This patch makes a few minor changes to the markup of the item detail
page in order for everything to be valid and styled correctly:

- Remove unnecessary and invalid <br clear="both"/>
- Move heading inside container so that floats clear correctly
- Change <span> tag back to <div> to enable the correct style.

To test, apply the patch and view the item details page for a title with
multiple items.

 - Each section starting with "Barcode" should be bordered above by a
   two-pixel gray line
 - The "statuses" section should be styled like the areas above and
   below, not as an ordered list.

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2020-07-31 08:56:49 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 6 Lucas Gass 2020-08-13 16:31:41 UTC
missing 20.05.x dependency, no backport