the WATCHDOG constants later to ERROR (e.g. CRITICAL, EMERG and ALERT) don't have any icons in the watchdog. It should at least default with the _ERROR icon.

CommentFileSizeAuthor
#8 dblog.css-reuse_error_icon-1024328-8.patch1.11 KBpillarsdotnet
#5 dblog.css-reuse_error_icon_on_higher_severity_levels-1024328-5.patch1.06 KBpillarsdotnet
#3 dblog.patch616 bytesdroplet

Comments

VikrantR’s picture

VikrantR commented

As it is database field 'severity' can not put any icon in it.

Sylvain Lecoy’s picture

Sylvain Lecoy commented

What do you mean by database field ?

No I'm speaking about the /admin/reports/dblog report.

droplet’s picture

droplet commented
Priority: Major » Normal
Status: Active » Needs review
StatusFileSize
new dblog.patch616 bytes
droplet’s picture

droplet commented
Version: 7.x-dev » 8.x-dev
Assigned: Unassigned » droplet
Issue tags: +Needs backport to D7

still working for both D7 & D8

pillarsdotnet’s picture

pillarsdotnet commented
Title: WATCHDOG_ > ERROR don't have any icons » Re-use message-16-error.png icon for dblog severity levels higher than LOG_ERR
StatusFileSize
new dblog.css-reuse_error_icon_on_higher_severity_levels-1024328-5.patch1.06 KB

Re-roll of #3 using git format-patch. No changes necessary, as it still applies.

pillarsdotnet’s picture

pillarsdotnet commented

#5: dblog.css-reuse_error_icon_on_higher_severity_levels-1024328-5.patch queued for re-testing.

aspilicious’s picture

aspilicious commented
Status: Needs review » Needs work 
+++ b/modules/dblog/dblog.cssundefined
@@ -52,7 +52,9 @@ table#admin-dblog td.icon {
+}
\ No newline at end of file

Newline gone?

Powered by Dreditor.

pillarsdotnet’s picture

pillarsdotnet commented
StatusFileSize
new dblog.css-reuse_error_icon-1024328-8.patch1.11 KB

Sure.

pillarsdotnet’s picture

pillarsdotnet commented
Status: Needs work » Needs review
pillarsdotnet’s picture

pillarsdotnet commented
Issue tags: -Needs backport to D7

#8: dblog.css-reuse_error_icon-1024328-8.patch queued for re-testing.

pillarsdotnet’s picture

pillarsdotnet commented
Issue tags: +Needs backport to D7

#8: dblog.css-reuse_error_icon-1024328-8.patch queued for re-testing.

droplet’s picture

droplet commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix
dries’s picture

dries commented

This seems like a sensible thing to do. Let's see if there is more feedback before I commit this.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Looks ready to fly for me.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Needs backport to D7

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

Issue tags: +Quick fix, +Needs backport to D7

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.