The context_ui_preprocess_page function is ignoring theme settings with regard to the display of the primary and secondary menus. The attached patch fixes this, although I didn't spend much time figuring out why this function was doing this in the first place, so it may not be the appropriate fix.

CommentFileSizeAuthor
#9 context-449816.patch1.36 KBsteven jones
#6 context.449816.patch1.79 KBjhedstrom
#5 context.449816.patch1.8 KBjhedstrom
#4 context.449816.patch3.67 KBjhedstrom
context_ui.patch1.67 KBjhedstrom

Comments

tahiticlic’s picture

tahiticlic commented

Same problem with the 2.x version.

Body classes I add are ignored, I've checked and the context_preprocess_page function is called normally and modifies correctly the $vars values, but at the end the body classes are set to their intial values...

[Edit] in my case, it was Skinr module which preprocess occured after this one I guess. Solved.[/Edit]

zoo33’s picture

zoo33 commented
Status: Needs review » Reviewed & tested by the community

Briefly tested and reviewed. Works and seems logical.

tahiticlic’s picture

tahiticlic commented

It works if you find the conflict... The use of the preprocess functions introduce such conflicts between modules, which individually does work fine. There's also a conflict with zen theme for instance.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Version: 6.x-1.x-dev » 6.x-2.0-beta7
StatusFileSize
new context.449816.patch3.67 KB

Here's a reroll against 2.x

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
StatusFileSize
new context.449816.patch1.8 KB

Sorry, that was a broken patch.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
StatusFileSize
new context.449816.patch1.79 KB

Still broken. Sorry for the spam. This one works against 2.x.

steven jones’s picture

steven jones commented
Status: Reviewed & tested by the community » Fixed

Committed to 6.x-2.x, thanks!

Status: Fixed » Closed (fixed)

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

steven jones’s picture

steven jones commented
Version: 6.x-2.0-beta7 » 6.x-3.0-beta4
Status: Closed (fixed) » Needs review
StatusFileSize
new context-449816.patch1.36 KB

This is an issue in context 3.x

Patch attached.

yhahn’s picture

yhahn commented
Assigned: Unassigned » yhahn
Priority: Normal » Critical
yhahn’s picture

yhahn commented
Version: 6.x-3.0-beta4 » 6.x-3.x-dev
yhahn’s picture

yhahn commented
Status: Needs review » Fixed

Thanks. http://drupal.org/cvs?commit=399594

Status: Fixed » Closed (fixed)

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