This is an easy one.

Please remove the redundant characters "table "
from line 23,.. "div.status, table tr.ok {" ..of
[d7core]/modules/system/system.messages.css
since it goes without saying that a table row (tr) is part of a table.

Please make the full rule declaration ...

div.status, table tr.ok {
  background-color: #F8FFF0;
}

In the case of dark background themes,
this will, among other things,
facilitate the implementation of themes imported from D6
since the declaration as it now stands overrides
the seemingly logical and simple use of "tr.ok" alone.

I feel strongly about this, and I am happy to expound
if you should desire further detailed reasoning.

Thanking you in advance...

-c

CommentFileSizeAuthor
#2 #palette .item-selected .jpg129.25 KBchristopher james francis rodgers
#3 palette .item-selected .jpg129.25 KBchristopher james francis rodgers
#1 table_tr.warning.JPG150.21 KBchristopher james francis rodgers
table_tr.ok_.JPG100.18 KBchristopher james francis rodgers

Comments

christopher james francis rodgers’s picture

christopher james francis rodgers commented
StatusFileSize
new table_tr.warning.JPG150.21 KB

Kindly ditto above:

Beginning line 36 of [d7.2-core]/modules/system/system.messages.css
please remove the characters "table "so that the full rule declaration is...

div.warning, tr.warning {
  background-color: fffce5;
}

Thanks again.

All the best to you all.

-c

christopher james francis rodgers’s picture

christopher james francis rodgers commented
Title: Minor CSS change in modules/system/system.messages.css to facilitate dark background theming and import. » Drupal 7 Module CSS "Rule Declarations" Must Specify "background-color" And "color" Instead of Only One or the Other.
StatusFileSize
new #palette .item-selected .jpg129.25 KB

Moved.

christopher james francis rodgers’s picture

christopher james francis rodgers commented
Title: Minor CSS change in modules/system/system.messages.css to facilitate dark background theming and import. » Drupal 7 Module CSS "Rule Declarations" Must Specify "background-color" And "color" Instead of Only One or the Other.
StatusFileSize
new palette .item-selected .jpg129.25 KB

* Moved.

christopher james francis rodgers’s picture

christopher james francis rodgers commented
Title: Drupal 7 Module CSS "Rule Declarations" Must Specify "background-color" And "color" Instead of Only One or the Other. » Minor CSS change in modules/system/system.messages.css to facilitate dark background theming and import.

Moved to http://drupal.org/node/1173774

lyricnz’s picture

lyricnz commented
Title: Drupal 7 Module CSS "Rule Declarations" Must Specify "background-color" And "color" Instead of Only One or the Other. » Minor CSS change in modules/system/system.messages.css to facilitate dark background theming and import.
Status: Active » Closed (won't fix)

Looks like this issue can be closed, as the original topic is was addressed in #1173774: Drupal 7 Module CSS "Rule Declarations" Should Always Specify "background-color" And "color" Instead of Only One or the Other. with a "won't fix"