Bug 17417

Summary: Correct invalid markup around news on the staff client home page
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: f.demians, julian.maurice, kyle, veron
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 17417 - Correct invalid markup around news on the staff client home page
Bug 17417 - Correct invalid markup around news on the staff client home page
Bug 17417 [Revised] Correct invalid markup around news on the staff client home page
[PASSED QA] Bug 17417 [Revised] Correct invalid markup around news on the staff client home page

Description Owen Leonard 2016-10-10 12:15:05 UTC
There are a couple of HTML validation errors in the area of the staff client home page template related to displaying news.
Comment 1 Owen Leonard 2016-10-10 12:17:54 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-10-10 19:03:24 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2016-10-14 15:00:05 UTC
Created attachment 56552 [details] [review]
Bug 17417 [Revised] Correct invalid markup around news on the staff client home page

This patch fixes two minor errors in the markup around displaying news
on the staff client home page: Adding a missing "class" attribute, and
changing the container around news items from <p> to <div> (because news
items might contain <p> and nesting them is incorrect).

To test, apply the patch and view the staff client home page with news
items showing. Validate the page and confirm that there are no errors.

Signed-off-by: Marc <veron@veron.ch>

Revision for QA removes an extra </div> from elsewhere in the template
to make the validator return no errors.
Comment 4 Katrin Fischer 2016-10-24 11:58:58 UTC
Created attachment 56745 [details] [review]
[PASSED QA] Bug 17417 [Revised] Correct invalid markup around news on the staff client home page

This patch fixes two minor errors in the markup around displaying news
on the staff client home page: Adding a missing "class" attribute, and
changing the container around news items from <p> to <div> (because news
items might contain <p> and nesting them is incorrect).

To test, apply the patch and view the staff client home page with news
items showing. Validate the page and confirm that there are no errors.

Signed-off-by: Marc <veron@veron.ch>

Revision for QA removes an extra </div> from elsewhere in the template
to make the validator return no errors.

Signed-off-by: Katrin Fischer  <katrin.fischer@bsz-bw.de>
Comment 5 Kyle M Hall 2016-10-28 11:46:00 UTC
Pushed to master for 16.11, thanks Owen!
Comment 6 Frédéric Demians 2016-11-02 18:19:03 UTC
Pushed in 16.05. Will be in 16.05.06.
Comment 7 Julian Maurice 2016-11-08 09:28:33 UTC
Pushed to 3.22.x, will be in 3.22.13