Small error in how the HTML is printed in page.tpl.php:

<div id="columns"<?php print $page['menu_bar'] ? 'class="no-menu-bar"' : '' ;?>><div class="columns-inner clearfix">

Should have a space after the id="columns", otherwise the attributes are glued together and generate invalid HTML:

<div id="columns" <?php print $page['menu_bar'] ? 'class="no-menu-bar"' : '' ;?>><div class="columns-inner clearfix">

I'll submit a patch. Tested against 7.x-2.x and 7.x-3.x branches.

CommentFileSizeAuthor
#1 0001-Issue-1623214-add-a-space-between-id-and-class-attri.patch829 bytesbgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bgm’s picture

bgm CreditAttribution: bgm commented
Status: Active » Needs review
FileSize
0001-Issue-1623214-add-a-space-between-id-and-class-attri.patch829 bytes

Here's a trivial patch against 7.x-3.x. Should apply to 7.x-2.x as well.

rudetrue’s picture

rudetrue CreditAttribution: rudetrue commented

subscribing..

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Sorry, this got fixed inadvertently along the way. Closing.