Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
On the dblog page at /admin/reports/dblog, the td's in the active column get a red (#eeb9b9) background color if the status of the log message is 'error'.
However, in Seven this only happens on even rows (see attached screenshot), because the background color on odd rows is reset to #f3f4ee on line 527 of style.css:
table tr.odd td.active {
background: #f3f4ee;
}
I'd love to submit a patch but I'm not sure which color it should be. Are there official design files for this?
Comment | File | Size | Author |
---|---|---|---|
#3 | 813516.patch | 671 bytes | asrob |
#2 | broken_warning.png | 23.71 KB | nevergone |
#1 | dblog-error-odd-F3CDCD.png | 12.84 KB | marcvangend |
dblog-error-odd.png | 15.99 KB | marcvangend |
Comments
Comment #1
marcvangendThe difference between the normal even/odd background colors in the active column is +13 Luminosity in the HSL color space. We can derive odd-row colors from that for the colors of all severity levels.
There is one complicating factor: Both the severity level and the zebra striping are coded as classes on the
<tr>
element. The cleanest way to apply the colors would be this:(See the attached screenshot for the result.) Unfortunately, this type of css selector (multiple class names) is not supported by IE6 and before.
Comment #2
nevergone CreditAttribution: nevergone commentedWarning message is broken:
Comment #3
asrobI fixed, please test it.
Comment #4
marcvangendThanks for reminding me, I forgot about this issue...
Asrob, I'm afraid we cannot use this fix, because we say that we support IE6 in core themes. I think we need extra classes.
Comment #5
Jeff Burnz CreditAttribution: Jeff Burnz commentedWell... IE6 support only has to be to the extent that its not broken, certainly doesn't need to be perfect. Bartik as a transparent png logo which looks horrid in IE6, and while this is a bit more serious since its error and warning messages etc I could live with the chained selectors. You could also give IE6 some brutal override so they were all one color (no odd/even striping). Put it this way - should we fix this now for the majority and leave a dying browser with something OK, or leave looking bad for everyone - cause time is running out :)
Comment #6
marcvangendThanks Jeff. I do consider the situation from the screenshot broken, because it's bad for usability. So, IMO it needs to be fixed for IE6 as well. But you're right, we can just introduce the nice styling for modern browsers and pick a single color for IE6 only. That way we don't need to add extra classes to the markup.
Comment #7
Jeff Burnz CreditAttribution: Jeff Burnz commentedOK sounds good, so we need a patch that:
- sets the nice styles like the patch in #3 for modern browsers using chained selectors
- add an override for IE6 to use just one color
Are there any other status's that we need to think about such as OK, Content, User - are they all looking right?
Comment #8
marcvangendJeff: yes that's what we need to do. We indeed need to check all types of message levels. While we're at it, let's check other core themes as well.