Posted by webchick on October 12, 2009 at 4:43am
Issue Summary
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.
Comments
#1
In themes/seven/style.css from line no 116 to 140 changing
#console div.messages {
padding: 9px;
margin: 1em 0;
color: #036;
background: #bdf;
border: 1px solid #ace;
}
#console div.warning {
color: #840;
background: #fe6;
border-color: #ed5;
}
#console div.error {
color: #fff;
background: #e63;
border-color: #d52;
}
#console div.status {
color: #360;
background: #cf8;
border-color: #be7;
}
to
div.messages {
padding: 9px;
margin: 1em 0;
color: #036;
background: #bdf;
border: 1px solid #ace;
}
div.warning {
color: #840;
background: #fe6;
border-color: #ed5;
}
div.error {
color: #fff;
background: #e63;
border-color: #d52;
}
div.status {
color: #360;
background: #cf8;
border-color: #be7;
}
Will fix this.
#2
Attached patch will fix this.
#3
patch 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.
#4
dang, screenshot fail, here we go
#5
Double 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.
#6
Awesome, thanks! :)
Committed to HEAD!
#7
Automatically closed -- issue fixed for 2 weeks with no activity.