fix(aria-menu): element with role="menu-item" must be children of a direct parentElement having role="menu", all elements between role="menu" and role="menu-item" must be presentation as role="none", see: https://accessibilityinsights.io/info-examples/web/aria-required-parent/

This commit is contained in:
Martin 2022-10-06 16:10:45 +02:00 committed by mfo
parent 9e399aa199
commit b358c505fa

View file

@ -9,37 +9,37 @@
%button.button.dropdown-button{ data: { menu_button_target: 'button' } } %button.button.dropdown-button{ data: { menu_button_target: 'button' } }
= t('views.users.dossiers.dossier_action.actions') = t('views.users.dossiers.dossier_action.actions')
.dropdown-content.fade-in-down{ data: { menu_button_target: 'menu' }, id: dom_id(dossier, :actions_menu) } .dropdown-content.fade-in-down{ data: { menu_button_target: 'menu' }, id: dom_id(dossier, :actions_menu) }
%ul.dropdown-items %ul.dropdown-items{ role: :none }
- if has_edit_action - if has_edit_action
- if dossier.brouillon? - if dossier.brouillon?
%li %li{ role: :none }
= link_to(url_for_dossier(dossier)) do = link_to(url_for_dossier(dossier)) do
%span.icon.edit %span.icon.edit
.dropdown-description .dropdown-description
= t('views.users.dossiers.dossier_action.edit_draft') = t('views.users.dossiers.dossier_action.edit_draft')
- else - else
%li %li{ role: :none }
= link_to modifier_dossier_path(dossier) do = link_to modifier_dossier_path(dossier) do
%span.icon.edit %span.icon.edit
.dropdown-description .dropdown-description
= t('views.users.dossiers.dossier_action.edit_dossier') = t('views.users.dossiers.dossier_action.edit_dossier')
- if has_transfer_action - if has_transfer_action
%li %li{ role: :none }
= link_to transferer_dossier_path(dossier) do = link_to transferer_dossier_path(dossier) do
%span.icon.person %span.icon.person
.dropdown-description .dropdown-description
= t('views.users.dossiers.dossier_action.transfer_dossier') = t('views.users.dossiers.dossier_action.transfer_dossier')
- if has_new_dossier_action - if has_new_dossier_action
%li %li{ role: :none }
= link_to procedure_lien(dossier.procedure) do = link_to procedure_lien(dossier.procedure) do
%span.icon.new-folder %span.icon.new-folder
.dropdown-description .dropdown-description
= t('views.users.dossiers.dossier_action.start_other_dossier') = t('views.users.dossiers.dossier_action.start_other_dossier')
- if has_delete_action - if has_delete_action
%li.danger %li.danger{ role: :none }
= link_to delete_dossier_dossier_path(dossier), method: :patch, data: { disable: true, confirm: "En continuant, vous allez supprimer ce dossier ainsi que les informations quil contient. Toute suppression entraîne lannulation de la démarche en cours.\n\nConfirmer la suppression ?" } do = link_to delete_dossier_dossier_path(dossier), method: :patch, data: { disable: true, confirm: "En continuant, vous allez supprimer ce dossier ainsi que les informations quil contient. Toute suppression entraîne lannulation de la démarche en cours.\n\nConfirmer la suppression ?" } do
%span.icon.delete %span.icon.delete
.dropdown-description .dropdown-description