Right now, status messages In Garland look pretty crappy, especially if differently styled status messages are shown together on one page.
The enclosed patch corrects the various margins and paddings both for LTR and RTL. Also, I improved the positioning of ul bullets (which are especially prominent in menu items).
I guess I don't have to be more detailed as the pictures say everything.

LTR has been tested in Firefox 2.0.0.11, IE 7 and Opera 9.25. RTL remains to be tested.

CommentFileSizeAuthor
#3 orange-warning.png17.41 KBpancho
#1 menu.png18.98 KBpancho
messages.png48.97 KBpancho
messages2.png51.98 KBpancho
messages3.png49.08 KBpancho
style_ul_and_status_messages.patch2.11 KBpancho

Comments

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
StatusFileSize
new menu.png18.98 KB

And here's a screenshot for the menu items.

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
Status: Needs review » Needs work

I like what your doing here...
But to make things even better, make the color of the warning a little less black.. Perhaps #333 is better for that, furthermore this is really nice...

If you could change the point I mentioned above, this is really RTBC! :-)

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
StatusFileSize
new orange-warning.png17.41 KB

Rather I'd use a dark orange (#F0C020) for warnings to match the styles of error and status messages. Attached is another screenshot. If I get some positive reviews for the dark orange, I'll reroll the patch with this.

damien tournoud’s picture

damien tournoud commented

+1 for the looks, with dark orange.

For the patch: note that there are already two declarations of ".messages" (once along with .preview) and a declaration of ".messages ul" in the style sheets. All this could and should be all in one place.

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
Title: Style crappy status messages and ul's » Garland: Status messages & ul's could be styled better
Version: 6.x-dev » 7.x-dev
Assigned: pancho » Stefan Nagtegaal

Assigning myself...

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

There is a patch for this at http://drupal.org/node/231115..
Please review it there, and report back to me (using that issue, not this one)..

I'll close this bug report once the patch http://drupal.org/node/231115 is in..

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
Status: Needs work » Closed (works as designed)

I investigated this, and I'm absolutely positive that the best way to color this is already in core.
I'm marking this "By Design"...

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
Status: Closed (works as designed) » Fixed

fixed by http://drupal.org/node/231115

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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