Here's a screenshot from the #560780: Add Image Field to image.module patch, which displays a status message inline:

Status message - Seven

I think this is a bug in the theme because in Garland it looks fine:

Status message - Garland

I am no kind of CSS expert, but I'm guessing this is pretty easy to fix. Tagging as Novice.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sivaji_Ganesh_Jojodae’s picture

Status: Active » Needs review
Issue tags: +Quick fix
FileSize
99.37 KB

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.

Sivaji_Ganesh_Jojodae’s picture

Attached patch will fix this.

arianek’s picture

patch applied cleanly to head. looks good to me, screenshot:

Only local images are allowed.

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.

arianek’s picture

FileSize
28.49 KB

dang, screenshot fail, here we go

rickvug’s picture

Status: Needs review » Reviewed & tested by the community

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.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Awesome, thanks! :)

Committed to HEAD!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Novice

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