Skip to content
Snippets Groups Projects

2942404 Messages should have role=status instead of role=contentinfo

Open Ben Mullins requested to merge issue/drupal-2942404:2942404-11 into 11.x
2 unresolved threads

Closes #2942404

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
23 23 #}
24 24 <div data-drupal-messages>
25 25 {% for type, messages in message_list %}
26 <div role="contentinfo" aria-label="{{ status_headings[type] }}"{{ attributes|without('role', 'aria-label') }}>
26 <div role="region" aria-label="{{ status_headings[type] }}"{{ attributes|without('role', 'aria-label') }}>
27 27 {% if type == 'error' %}
28 28 <div role="alert">
29 {% elseif type == 'status' or type == 'warning' %}
30 <div role="status">
  • 39 41 {% else %}
    40 42 {{ messages|first }}
    41 43 {% endif %}
    42 {% if type == 'error' %}
    44 {% if type in ['error', 'status', 'warning'] %}
  • This makes a lot of sense.

  • Please register or sign in to reply
    Loading