Bug 8522

Summary: Markup errors cause problems with customized CSS
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: chris, mtj, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Screenshot showing the header background problem
Bug 8522 - Markup errors cause problems with customized CSS
Bug 8522 - Markup errors cause problems with customized CSS

Description Owen Leonard 2012-07-30 14:40:50 UTC
Created attachment 11198 [details]
Screenshot showing the header background problem

A few files have markup errors which are causing display problems when a custom stylesheet is set up to override the background of the #header region.
Comment 1 Owen Leonard 2012-07-30 15:01:14 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-07-30 16:38:51 UTC
Sorry, the CSS example didn't come through because it was prefixed with "#" !

#header { background : #336699 none; }
Comment 3 Chris Cormack 2012-08-07 03:54:14 UTC
Created attachment 11396 [details] [review]
Bug 8522 - Markup errors cause problems with customized CSS

Some includes and templates contained duplicate "header" ids in
the markup. The problem should have come up in routine page
validation, but was obvious when custom CSS was applied.

To test, load any of the affected pages and validate the
generated HTML. There should be no errors about 'ID "header"
already defined." Or, add custom CSS to intranetusercss:

...and confirm that only the topmost menu background is affected.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 4 Mason James 2012-08-10 07:02:30 UTC
(In reply to comment #3)
> Created attachment 11396 [details] [review]
> Bug 8522 - Markup errors cause problems with customized CSS
> 
> Some includes and templates contained duplicate "header" ids in
> the markup. The problem should have come up in routine page
> validation, but was obvious when custom CSS was applied.
> 
> To test, load any of the affected pages and validate the
> generated HTML. There should be no errors about 'ID "header"
> already defined." Or, add custom CSS to intranetusercss:
> 
> ...and confirm that only the topmost menu background is affected.
> 
> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

patch look good, passing QA


mason@xen1:~/git/head$ koha-qa.pl
        * e6a5599 Bug 8522 - Markup errors cause problems with customized CSS
                koha-tmpl/intranet-tmpl/prog/en/includes/budgets-admin-search.inc
                koha-tmpl/intranet-tmpl/prog/en/includes/cities-admin-search.inc
                koha-tmpl/intranet-tmpl/prog/en/includes/patrons-admin-search.inc
                koha-tmpl/intranet-tmpl/prog/en/includes/roadtype-admin-search.inc
                koha-tmpl/intranet-tmpl/prog/en/includes/z3950-admin-search.inc
                koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt

- perlcritic-progressive tests... OK
- perl -c syntax tests... OK
- xt/tt_valid.t tests... OK
- xt/author/valid-template.t tests... OK
mason@xen1:~/git/head$
Comment 5 Chris Cormack 2012-08-11 19:18:32 UTC
Pushed to 3.8.x will be in 3.8.4
Comment 6 Paul Poulain 2012-08-31 16:53:39 UTC
Patch pushed to master. Already on stable, setting status accordingly