A stretch to call this a bug report, but...

An empty <div id="messages"> is written when no messages are inside it. I've had a few styling problems that were easier to solve with the #messages <div> omitted unless there are messages to display. Patch below.

CommentFileSizeAuthor
#1 omit_empty_messages_div-1988736-1.patch1.88 KBmshepherd

Comments

mshepherd’s picture

mshepherd commented
Status: Active » Needs review
StatusFileSize
new omit_empty_messages_div-1988736-1.patch1.88 KB

Patch attached for review

Jeff Burnz’s picture

Jeff Burnz commented

Now, how did I miss this one for so long, will review tomorrow!

Jeff Burnz’s picture

Jeff Burnz commented

OK, we can get away with this, but it seems like a hack, that its a bug in Drupal core rendering system, which is OK since we often work around such issues and have done all along in D7.

Jeff Burnz’s picture

Jeff Burnz commented
Issue summary: View changes
Status: Needs review » Fixed

Fixed! Thank-you for the patch :)

Status: Fixed » Closed (fixed)

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