See screenshot.

Stuff.

CommentFileSizeAuthor
#3 684268-toolbar-preview-gap.patch610 bytesseutje
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cosmicdreams’s picture

cosmicdreams CreditAttribution: cosmicdreams commented

Does this issue go away once you tell drupal to clear cache or refresh your browser's cache? I think I ran into this issue last week.

seutje’s picture

seutje CreditAttribution: seutje commented

looks like the body is improperly getting the "toolbar" and "toolbar-drawer" classes, hmz...

seutje’s picture

seutje CreditAttribution: seutje commented
Component: Seven theme » toolbar.module
Status: Active » Reviewed & tested by the community
FileSize
684268-toolbar-preview-gap.patch610 bytes

check for the presence of the toolbar before adding the classes?

seems more toolbar.module related

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

I saw someone with the same patch :)...
Gonna crosslink...

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

Same patch: #715762: Toolbar classes added even when not on page

casey’s picture

casey CreditAttribution: casey commented

Marked the other one duplicate. This one is still RTBC.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

seutje’s picture

seutje CreditAttribution: seutje commented
Status: Fixed » Reviewed & tested by the community

ups, didn't mean to RTBC it, just meant to set it to needs review :x

and sry for not stumbling on #715762: Toolbar classes added even when not on page, was just looking at seven's issues

seutje’s picture

seutje CreditAttribution: seutje commented
Status: Reviewed & tested by the community » Fixed

swear to god, every single time >_<

Status: Fixed » Closed (fixed)

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