Commit 7b2531c3 authored by catch's avatar catch
Browse files

task: #3572169 Put the mobile sidebar close button into the TAB sequence 

By: kentr
By: rkoller
By: mherchel
By: mgifford
By: katannshaw
(cherry picked from commit 21329825)
parent ba10765d

core/modules/navigation/layouts/navigation.html.twig

+2 −1
Original line number Diff line number Diff line
@@ -71,9 +71,10 @@ 
          icon: { icon_id: 'arrow-left' },

          text: 'Back'|t,

        } only %}

        {# The close button should be in the tab sequence. Therefore, `tabindex` is omitted here. #}

        {% include 'navigation:toolbar-button' with {

          action: 'Collapse sidebar'|t,

          attributes: create_attribute({ 'aria-controls': 'admin-toolbar', 'tabindex': '-1', 'type': 'button' }),

          attributes: create_attribute({ 'aria-controls': 'admin-toolbar', 'type': 'button' }),

          extra_classes: ['admin-toolbar__close-button'],

          icon: { icon_id: 'cross' },

        } only %}