Enabling Mobile Plugin prevents this module from working.

Disable Mobile Plugin and this works as expected.

CommentFileSizeAuthor
#8 dont-break-custom-theme-803866-8.patch916 bytesmikl
#4 admin_theme.patch763 bytesdavyvdb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davyvdb’s picture

davyvdb CreditAttribution: davyvdb commented
Category: bug » support
Status: Active » Fixed

Giving the admin_theme module a higher weight should fix the issue.

Status: Fixed » Closed (fixed)

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

nrackleff’s picture

nrackleff CreditAttribution: nrackleff commented
Status: Closed (fixed) » Active

I updated the weight of the module and now the admin theme works but the mobile theme does not show up anymore. Is there a way to make these two play nice together?

davyvdb’s picture

davyvdb CreditAttribution: davyvdb commented
FileSize
admin_theme.patch763 bytes

Here's a patch that should fix this by checking if the $custom_them is the admin theme.

davyvdb’s picture

davyvdb CreditAttribution: davyvdb commented
Status: Active » Needs review
davyvdb’s picture

davyvdb CreditAttribution: davyvdb commented
Title: Doesn't work with mobile plugin » Doesn't work with modules that also alter $custom_theme
nrackleff’s picture

nrackleff CreditAttribution: nrackleff commented

Thank you for creating the patch. We have actually disabled the mobile version of the site for now, so I am not able to test at this time. I hope to bring it back soon.

mikl’s picture

mikl CreditAttribution: mikl commented
FileSize
dont-break-custom-theme-803866-8.patch916 bytes

I've added a patch that adresses this, that also works with Drush make. It also adresses a small issue raised in #829352: Incompatiblity with Custom Theme modules.

justy’s picture

justy CreditAttribution: justy at TrazerEdge commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Closing this old Drupal 6 issue.