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.
Here's a screenshot from the #560780: Add Image Field to image.module patch, which displays a status message inline:
I think this is a bug in the theme because in Garland it looks fine:
I am no kind of CSS expert, but I'm guessing this is pretty easy to fix. Tagging as Novice.
Comment | File | Size | Author |
---|---|---|---|
#4 | imgmessage.jpg | 28.49 KB | arianek |
#2 | 601972-status-message-not-styled-1.patch | 1009 bytes | Sivaji_Ganesh_Jojodae |
#1 | Screenshot-1.png | 99.37 KB | Sivaji_Ganesh_Jojodae |
Comments
Comment #1
Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commentedIn themes/seven/style.css from line no 116 to 140 changing
to
Will fix this.
Comment #2
Sivaji_Ganesh_Jojodae CreditAttribution: Sivaji_Ganesh_Jojodae commentedAttached patch will fix this.
Comment #3
arianek CreditAttribution: arianek commentedpatch applied cleanly to head. looks good to me, screenshot:
going to leave this open for a code review yet, i think rickvug is going to have a look at it shortly before we RTBC it.
Comment #4
arianek CreditAttribution: arianek commenteddang, screenshot fail, here we go
Comment #5
rickvug CreditAttribution: rickvug commentedDouble checked this and the changes look good to me. The code comment change to switch the heading from "console" to "messages" makes a whole lot more sense also. RTBC.
Comment #6
webchickAwesome, thanks! :)
Committed to HEAD!