In #1986408: Message style update we updated the visual styling of messages to match the #1986434: New visual style for Seven. We decided the leave the icons to a follow up.

Some Libricons are now in core, instructions of how to add and implement them are in #2032773: Use Libricons (icon font) in Seven, consider using it more broadly in core

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bojhan’s picture

FileSize
102.7 KB

Here you go :)

AdamGerthel’s picture

Check, error and warning icons are already included and available in Libricons aren't they?

Nevermind, they're not colored there

AdamGerthel’s picture

Status: Active » Needs review
FileSize
6.72 KB

Here's a first attempt at adding this. It's my first patch so bare with me.

AdamGerthel’s picture

Hmm, perhaps the old files should be removed as well?

I've searched through core to find any other references to those files and found the following:

message-16-ok.png
- No uses anywhere (after my patch)

message-16-warning.png
- Two uses.

  1. Line 43 in core/modules/dblog/css/dblog.module.css
  2. Line 220 in core/modules/system/css/system.admin.css

message-16-error.png
- Four uses.

  1. Line 49 in core/modules/dblog/css/dblog.module.css
  2. Line 1430 in core/modules/filter/filter.module
  3. Line 87 in core/modules/filter/lib/Drupal/filter/Tests/FilterHtmlImageSecureTest.php
  4. Line 217 in core/modules/system/css/system.admin.css
LewisNyman’s picture

Status: Needs review » Needs work

Looking good, I think you might want to run the svgs through something like SVGO to remove some of the redundant meta data. Let's remove the icons that are not used and I suppose we'll have to open up a separate issue to replace the icons that are referenced in other files in core.

AdamGerthel’s picture

Here's a new patch. Metadata removed from the SVG's and deleted message-16-ok.png that isn't used anymore.

AdamGerthel’s picture

Status: Needs work » Needs review
Bojhan’s picture

This probably needs some screenshots?

AdamGerthel’s picture

FileSize
15.88 KB
15.99 KB
57.99 KB
6.24 KB

Fixed SVG sizes (I had made them 32x32 by mistake in my last patch). Screenshots attached.

AdamGerthel’s picture

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

Looks awesome!

LewisNyman’s picture

Code looks good. RTBC++

Thanks Adam! The icons look so much better.

Committers, please credit Ry5n :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Wim Leers’s picture

Yay :)

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