With the updates made to the new page tpl code: http://drupal.org/node/367299 it became clear to me that some things needed to be moved, specifically $messages and $breadcrumb. Currently these reside in the main content area, i.e. above $content, within the same containing div. There've been a number of different conversations on moving these, ultimately we decided to not hash it out in favor of getting our new html into core more quickly, however I feel very strongly that we should move $messages and $breadcrumb to but within their own (individual) divs under the current navigation, above the main-wrapper. It makes little or no sense to separate the title from the content with these items, and I personally believe this is a remnant of the garland theme. I'm not sure how else it's embedded itself into our default output.

Garland is actually rather unique in the ways and reasons it does certain things. For the stylized approach garland takes it makes perfect sense to place these items within the content area, the breadcrumb is limited in size by the content area, there's no other place for messages but the content area... this is NOT the normal for designs, and as such I really feel we need to say "This is a garland specific setting, and it's great for garland, but our default html doesn't need to work this way.".

Content area currently restricts the size of the messages. These could be larger and draw more attention to themselves than they do currently if they were taken out of the content area. This would result (imo) in greater visibility.

Breadcrumbs are really their own thing as well and should be visually separated from content. They're closer in relation to navigation than anything else, and again I really think we should move all of these out of the content area.

Eclipse

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wretched sinner - saved by grace’s picture

geerlingguy’s picture

Do you think we should put them into their own div, on the same level as header and main?

starbow’s picture

I hope this ends up wrapping the print $messages with a div and an id. Makes it easier to grab them with jQuery and add effects.

EclipseGc’s picture

FileSize
59.95 KB

Webchick asked for a screenshot example. You can see how much more area could easily be devoted to messages, and the breadcrumbs are up with the rest of the navigation (ok, I don't have any in this theme, but you get the idea).

JohnAlbin’s picture

subscribe.

EclipseGc’s picture

FileSize
78.47 KB

As opposed to: (super rough but gets the idea across)

webchick’s picture

subscribe.

BrightLoudNoise’s picture

subscribe

EclipseGc’s picture

Issue tags: +Usability

Adding tag

Jeff Burnz’s picture

FileSize
140.63 KB

Just throwing in another screenshot. The proposal matches how I am doing things these days. From a development theme I am currently working on:

yoroy’s picture

I'm in favor of this as well. Who'll write the actual patch? :-)

Jeff Burnz’s picture

Ignore... sigh, not keeping up with the play here.

Jeff Burnz’s picture

FileSize
152.13 KB

Ignore this also, jeff needs a break :/

Jeff Burnz’s picture

Status: Needs review » Active
Issue tags: -Favorite-of-Dries
FileSize
1.27 KB

Patch to get the ball rolling.

yoroy’s picture

Status: Active » Needs review

status

Dries’s picture

Issue tags: +Favorite-of-Dries

Makes sense to me, and sounds like it could be a small UX improvement. Thoughts?

mortendk’s picture

it makes a lot of sence to move em
but im only a king so what do i know ;)

catch’s picture

I usually end up moving these out. Only once a year when I have to do some theming, but still, +1.

bsmirnov’s picture

I agree makes sense.

Jeff Burnz’s picture

Status: Active » Needs review
Issue tags: +Favorite-of-Dries

Note that unlike EclipseGc first screenshot, the patch does not move help, only breadcrumb and messages.

Not sure if we want to move help also, I tend to think this makes sense coming under the title.

yoroy’s picture

Status: Needs review » Reviewed & tested by the community

Good to make that point. Let's leave help out of the discussion here and just do what the issue title says:

Move $messages and $breadcrumb out of the content div and into the page div :)

so, RTBC? Patch looks simple enough

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Great. Committed to CVS HEAD.

Do we need to update any of the core themes?

Jeff Burnz’s picture

There's only Stark which is covered by this patch, and Garland is best left how it is I would think.

EclipseGc’s picture

I absolutely agree, stark is the only theme that matters here. Garland I think is the culprit who introduced this to begin with, so we're really just cleaning out garland code that should never have been for general consumption.

Eclipse

PS: thx jmburnz for pushing this one through. I got side tracked :-D

Status: Fixed » Closed (fixed)
Issue tags: -Favorite-of-Dries, -Usability

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