The following line in table.tpl.php:

$attributes['class'] .= ' sticky-enabled';

Causes havoc in the theme. It makes all node titles sticky, so it looks like the headings follow you down the page as you go. Only with Javascript enabled, of course.

I don't mean to be impolite, but are you aware that you shouldn't have all your logic in .tpl.php files? This theme breaks many, many Drupal coding conventions and standards and it's pretty ugly in there!

CommentFileSizeAuthor
#1 goofy-sticky_header-413568-1.patch699 bytesMark Theunissen

Comments

Mark Theunissen’s picture

Mark Theunissen commented
StatusFileSize
new goofy-sticky_header-413568-1.patch699 bytes
fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs review » Active

There is indeed a problem with sticky headers and some uses of theme_table.

Beyond this, believe it or not, there is significantly less "logic" in the templates than there used to be when I started evolving it.

It *is* indeed rather ugly code-wise, though, and I expect this to improve over time, but due to its bizarre construction of tables with additional rows and columns to create the borders, it is extremely unlikely it will really be clean some day. This is an early Drupal theme being ported, not a modern construction, and it shows.

Mark Theunissen’s picture

Mark Theunissen commented

Ok great! No offence meant. I'm not blaming anyone for the code, I just wanted to make sure that somebody was aware of it's quality... ;)