Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
dblog.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Sep 2013 at 11:45 UTC
Updated:
29 Jul 2014 at 22:53 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
klonos...less vague title (there are at least 3 issues with the same name: #2075949-10: Add Libricons to messages).
Comment #2
lewisnymantaggin
Comment #3
lewisnymanWorth having a quick look over the colours used, to see if we can align them with messages
Comment #4
outi commentedI made a patch with the svg icons we actually have.
By taking a look over the colors, do you mean the icon colors should be changed to match the message colors or the opposite? Anyway, the error icon color is now #ea2800 and the error message (border) color is #e62600. The warning icon color is #e29700 and the warning message (border) color is #e09600.
Comment #5
outi commentedComment #6
rteijeiro commentedTested the patch and seems good for me. (See screenshot)
SVG icons are loaded and also tried the no-svg fallback in IE8 and works well.
It's a RTBC for me.
Comment #7
lewisnymanIt was a bad idea to add colours to this issue, it's covered in #2070323: Apply seven style guide to dblog. The icons patch is RTBC.
Comment #8
webchickCommitted and pushed to 8.x. Thanks!
Comment #9.0
(not verified) commentedadded more info