Wrap messages in an own #messages DIV in default page.tpl.php

It is already possible to theme the messages with the div that gets added in theme_status_messages.

function theme_status_messages($variables) {
  $display = $variables['display'];
  $output = '';

  $status_heading = array(
    'status' => t('Status message'), 
    'error' => t('Error message'), 
    'warning' => t('Warning message'),
  );
  foreach (drupal_get_messages($display) as $type => $messages) {
    $output .= "<div class=\"messages $type\">\n";
    if (!empty($status_heading[$type])) {
      $output .= '<h2 class="element-invisible">' . $status_heading[$type] . "</h2>\n";
    }
    if (count($messages) > 1) {
      $output .= " <ul>\n";
      foreach ($messages as $message) {
        $output .= '  <li>' . $message . "</li>\n";
      }
      $output .= " </ul>\n";
    }
    else {
      $output .= $messages[0];
    }
    $output .= "</div>\n";
  }
  return $output;
}

Attached patch however implements the suggestion made above.

Not a bug and way to late. Core template review was over a year ago (so long ago I have forgotten when).

#1: 882796_page.tpl_.php_.patch queued for re-testing.

Lovely.

This will need documentation in the theme upgrade docs, tagging as such.

Committed to 8.x. Moving to 7.x for Angie to consider.

While this seems like a harmless enough fix, it would introduce a markup change in D7 themes, which could affect CSS-only themes. Since #1 has a workaround for D7, I'm inclined to keep this D8-only.

Marking back to 8.x-dev and "needs work" pending a http://drupal.org/node/add/changenotice node.

And since adding that summary is relatively easy, tagging as Novice.

My first Change record... Does this need more elaboration?

http://drupal.org/node/1294560

Looks good to me. Thanks!