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.

Support from Acquia helps fund testing for Drupal Acquia logo

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
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

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

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.