Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Follow-up to: #1838114: Change node form’s vertical tabs into details elements
Problem
- Any HTML element that has a
.statusclass is styled with an icon (intended for status messages).
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | drupal8.css-messages.1-before.png | 73.23 KB | andymartha |
| #5 | drupal8.css-messages.1-after.png | 79.07 KB | andymartha |
| #1 | drupal8.css-messages.1.patch | 1.13 KB | sun |
Comments
Comment #1
sunComment #3
jwilson3I completely agree with this change, and want to see more intelligent uses of adjoining css classes in core. CSSLint and IE6 be damned.
Not sure why this failed.. so give it another run...
Comment #4
sun#1: drupal8.css-messages.1.patch queued for re-testing.
Comment #5
andymartha CreditAttribution: andymartha commentedI can confirm that on a fresh installation of Drupal 8.x-dev from March 6th, 2013
1) creating a custom module to add a css class of status to a div or every div results in the checkmark icon being adding to the div
2) applying the patch drupal8.css-messages.1.patch in #1 by sun, then creating a custom module to add a css class of status to a div or every div results in a plain div while the status messages from Drupal core (eg drupal_set_message) still display the checkmark icon of status.
Please see screenshots. Not sure of the use case of this but, following the steps, it is unwanted behaviour fixed in my opinion.
Comment #6
Dries CreditAttribution: Dries commentedMost arty bug ever. ;-)
Committed to 8.x. Thanks!
Comment #7
webchickI think the award for Best Screenshot In The World Ever goes to andymartha! :D