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.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | omit_empty_messages_div-1988736-1.patch | 1.88 KB | mshepherd |
Comments
Comment #1
mshepherd CreditAttribution: mshepherd commentedPatch attached for review
Comment #2
Jeff Burnz CreditAttribution: Jeff Burnz commentedNow, how did I miss this one for so long, will review tomorrow!
Comment #3
Jeff Burnz CreditAttribution: Jeff Burnz commentedOK, 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.
Comment #4
Jeff Burnz CreditAttribution: Jeff Burnz commentedFixed! Thank-you for the patch :)