This patch is a follow up to the recent Garland cleanup patch described here: http://www.lullabot.com/articles/theming-best-practices-garland-gets-a-c...

CommentFileSizeAuthor
#4 lightfantastic_cleanup-264182-4.patch989 bytesmustafau
lightfantastic_cleanup.patch7.06 KBmustafau
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jbrauer’s picture

jbrauer CreditAttribution: jbrauer commented
Status: Needs review » Needs work

Hmmm this breaks upon installing. I'm working on re-factoring the concepts.

mustafau’s picture

mustafau CreditAttribution: mustafau commented

You should disable and enable the theme after applying the patch.

jbrauer’s picture

jbrauer CreditAttribution: jbrauer commented
Status: Needs work » Fixed

Applied most of the patch and fixed. Body tag remains as it was as this was causing some problems!

Thanks!

mustafau’s picture

mustafau CreditAttribution: mustafau commented
FileSize
lightfantastic_cleanup-264182-4.patch989 bytes

Following line appears twice in _preprocess_page():

$vars['tabs2'] = menu_secondary_local_tasks();
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.