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
PASSED: [[SimpleTest]]: [MySQL] 33,651 pass(es). View
#5 dblog.css-reuse_error_icon_on_higher_severity_levels-1024328-5.patch1.06 KBpillarsdotnet
PASSED: [[SimpleTest]]: [MySQL] 30,017 pass(es). View
#3 dblog.patch616 bytesdroplet
PASSED: [[SimpleTest]]: [MySQL] 31,501 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

VikrantR’s picture

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

Sylvain Lecoy’s picture

What do you mean by database field ?

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

droplet’s picture

Priority: Major » Normal
Status: Active » Needs review
FileSize
616 bytes
PASSED: [[SimpleTest]]: [MySQL] 31,501 pass(es). View
droplet’s picture

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

Title: WATCHDOG_ > ERROR don't have any icons » Re-use message-16-error.png icon for dblog severity levels higher than LOG_ERR
FileSize
1.06 KB
PASSED: [[SimpleTest]]: [MySQL] 30,017 pass(es). View

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

pillarsdotnet’s picture

aspilicious’s picture

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

FileSize
1.11 KB
PASSED: [[SimpleTest]]: [MySQL] 33,651 pass(es). View

Sure.

pillarsdotnet’s picture

Status: Needs work » Needs review
pillarsdotnet’s picture

pillarsdotnet’s picture

droplet’s picture

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

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

sun’s picture

Looks ready to fly for me.

Dries’s picture

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.