The H2 header for this theme contains an error which causes page titles to be too small

if ($title) { print '<h2'. ($tabs ? ' class="pagetitle with-tabs"' : 'pagetitle') .'>'. $title .'</h2>'; }
needs to be changed to
if ($title) { print '<h2'. ($tabs ? ' class="pagetitle with-tabs"' : ' class="pagetitle"') .'>'. $title .'</h2>'; }

the ?: (ternary?) expression causes pagetitle to be appended to the h2, creating a new html flag. This also screws up validation on an otherwise fine page. I'd make a patch but the fix is so simple that one's unnecessary. I'd do it myself if I had access to the code :-)

CommentFileSizeAuthor
#2 page.tpl_.patch755 bytesillogic-al

Comments

illogic-al’s picture

Title: Title » Header Title on pages messed up due to php code error.
illogic-al’s picture

Status: Active » Needs review
StatusFileSize
new755 bytes

OK, I decided to make a patch. Good thing too because i forgot to say where the error takes place. :-)
It's in the page.tpl.php file. Also the error only shows up when anonymous users view the page, or when users other than the one who created the page views it.

andrewbenkard’s picture

a big New Jersey THANK YOU to illogic-al.

Kilton Hopkins’s picture

I second that.

A big Chicago thank-you for illogic-al. You saved me time and that is invaluable.

toddgee’s picture

agree that this needs to be fixed (just came here to report same issue). Looks like this has been long-outstanding. I'll give the owner a ping.

d.cox’s picture

Ditto.