I don't understand why node/%/edit uses admin theme, but node/%/revision does not. Makes no sense except for revision views.

Patch coming.

CommentFileSizeAuthor
#1 934790-theme-node.patch1.46 KBagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 934790-theme-node.patch1.46 KB
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Title: Use _noide_custom_theme for revision callbacks » Use _node_custom_theme for revision callbacks

Fixes title typo.

damien tournoud’s picture

damien tournoud commented
Version: 7.0-beta1 » 7.x-dev
Status: Needs review » Reviewed & tested by the community

Makes sense, especially because those pages are served in the Overlay, and the change of theme is really weird there.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

I filed this bug against some other modules, too, such as NodeQueue, so perhaps we need some documentation about the best practice.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -#d7ux

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

Issue tags: +#d7ux

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.