Issue #2407743 by mortendk, danquah, rteijeiro, davidhernandez, sivaji@knackforge.com: Remove classes from system templates s*.html.twig

8.0.x
Alex Pott 2015-02-17 00:19:24 +00:00
parent 2bbeb8f3a8
commit 00df5bad02
2 changed files with 7 additions and 11 deletions

View File

@ -25,6 +25,10 @@ class MessageTest extends KernelTestBase {
* Tests setting messages output. * Tests setting messages output.
*/ */
function testMessages() { function testMessages() {
// Enable the Classy theme.
\Drupal::service('theme_handler')->install(['classy']);
$this->config('system.theme')->set('default', 'classy')->save();
drupal_set_message('An error occurred', 'error'); drupal_set_message('An error occurred', 'error');
drupal_set_message('But then something nice happened'); drupal_set_message('But then something nice happened');
$messages = array( $messages = array(

View File

@ -26,13 +26,7 @@
*/ */
#} #}
{% for type, messages in message_list %} {% for type, messages in message_list %}
{% <div class="{{ attributes.class }}" role="contentinfo" aria-label="{{ status_headings[type] }}">
set classes = [
'messages',
'messages--' ~ type,
]
%}
<div class="{{ attributes.addClass(classes).class }}" role="contentinfo" aria-label="{{ status_headings[type] }}">
{% if type == 'error' %} {% if type == 'error' %}
<div role="alert"> <div role="alert">
{% endif %} {% endif %}
@ -40,9 +34,9 @@
<h2 class="visually-hidden">{{ status_headings[type] }}</h2> <h2 class="visually-hidden">{{ status_headings[type] }}</h2>
{% endif %} {% endif %}
{% if messages|length > 1 %} {% if messages|length > 1 %}
<ul class="messages__list"> <ul>
{% for message in messages %} {% for message in messages %}
<li class="messages__item">{{ message }}</li> <li>{{ message }}</li>
{% endfor %} {% endfor %}
</ul> </ul>
{% else %} {% else %}
@ -52,6 +46,4 @@
</div> </div>
{% endif %} {% endif %}
</div> </div>
{# Remove type specific classes. #}
{{ attributes.removeClass(classes) }}
{% endfor %} {% endfor %}