fix(aria-allowed-role): landmark roles banner on header and contentinfo on footer to only report on top-level rule by Zidious · Pull Request #3142 · dequelabs/axe-core

Zidious · 2021-08-31T20:36:17Z

landmark roles banner on header and contentinfo on footer to report only top-level rule and fail aria-allowed-role when allowImplicit: false.

Clean up get-unallowed-roles to fail aria-role and not all roles.

Closes Issue: #3108 & #3213

Closes Issue: #3108

################ ################### #################### ####################### ####### ##### ###### ########################### ####### ################ ############################# ###### ################ ############################### ###### ################ ################################# ###### ################## ################################### ###### ############################### ############ ##### ############### ########## ######## ##### ############## ####### ####### ##### ############# ##### ##### ##### ############## #### #### ##### ############# #### #### ##### ############## #### #### ##### ############# #### #### ###### ############# ##### ##### ###### ############## ####### ####### ####### ############# ######## ######## ######## ############# ############ ############ ########## ############### ################################### ############################## ################################# ############################# ################################ ########################### ############################### ######################### ########################### ## #### ##################### ####################### ### ###### ################# ################### ############ ############ ########### ########### ########## ####### #### #### #### #### #### ###### ###################################################### ########## ###################################################### ############ ###################################################### ############## ###################################################### ############## ###################################################### ############## ###################################################### ############ ###################################################### ########## ###################################################### ###### #### #### #### #### #### ######################################################## ############################################################ ############################################################## ################################################################ ################################################################# ################################################################## ################################################################### ################################################################### ########### #### ######### #### ######## #### ####### #### ###### ##### ##### #### ### ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# ############# ######### ######### ####### ####### ###### ###### ##### ##### ##### ##### #### #### #### #### #### #### #### #### #### #### ### #### #### #### ### #### #### ###### #### ####### #### ######## ##### ####### ##### #### #### # ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# ############# ######### ######### ####### ####### ###### ###### ##### ##### ##### ##### #### #### #### #### #### #### #### #### ##### ##### ##### ##### ##### ##### ###### ###### ####### ####### ######### ######### ############# ############# ############################################## ############################################ ######################################## #################################### ################################ ############################ ################## #### #### #### #### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### #### #### #### #### #### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### #### #### #### #### #### ###### ###################################################### ########## ###################################################### ############ ###################################################### ############## ###################################################### ############## ###################################################### ############## ###################################################### ############ ###################################################### ########## ###################################################### ###### #### #### #### #### #### ######################################################## ############################################################ ############################################################## ################################################################ ################################################################# ################################################################## ################################################################### ################################################################### ########### #### ######### #### ######## #### ####### #### ###### ##### ##### #### ### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### #### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### ##### #### ##### ##### ##### #### ####### #### ######### ###################################################### ###################################################### ###################################################### ###################################################### ##################################################### #################################################### ################################################### ################################################ #### #### ######### ############## ################## ###################### ################ ## ############## ########### ######### ###### ######## ################ #################### ######################## ##### ########################## ########## ############################ ############# ############################## ############## ######### ######### ############### ######## ######## ############## ####### ####### #### ###### ###### ###### ##### ###### ###### #### ###### ###### #### ##### ##### #### ##### ##### ##### ##### ##### ###### ##### ##### ####### ###### ###### ######### ################################################ ################################################# ################################################# ################################################# ################################################# ############################################### ############################################# ######################################### ###### #### ### ## ###################### ############################## #################################### ######################################## ############################################ ############################################## ################################################ ############### ############### ########### ########### ######## ######## ####### ####### ###### ###### ##### ##### ##### ##### ##### ##### #### #### ##### ##### ##### ##### ##### ##### ##### ##### #### ##### ##### #### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### #### #### ###################### ############################## #################################### ######################################## ############################################ ############################################## ################################################ ############### ############### ########### ########### ######## ######## ####### ####### ###### ###### ##### ##### ##### ##### ##### ##### #### #### ##### ##### ##### ##### ##### ##### ##### ##### #### ##### ##### #### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### #### #### #### #### #### #### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### #### ##### #### ##### ##### ##### ###### ###### ###### ####### ####### ######## #### ######### #### ############ ###################################################### ##################################################### ##################################################### #################################################### ################################################### ################################################# ############################################## ########################################## #### #### ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# #### ############# ######### #### ######### ####### #### ####### ###### #### ###### ##### #### ##### ##### #### ##### #### #### #### #### #### #### #### #### #### #### #### #### #### #### ##### ### #### ##### #### #### ###### ### #### ####### #### #### ######### #### #### ############# #### ######################### ##### ######################## ##### ###################### #### #################### ################## ################ ########### ######## ################ #################### ######################## ##### ########################## ########## ############################ ############# ############################## ############## ######### ######### ############### ######## ######## ############## ####### ####### #### ###### ###### ###### ##### ###### ###### #### ###### ###### #### ##### ##### #### ##### ##### ##### ##### ##### ###### ##### ##### ####### ###### ###### ######### ################################################ ################################################# ################################################# ################################################# ################################################# ############################################### ############################################# ######################################### ###### #### ### ## ###################### ############################## #################################### ######################################## ############################################ ############################################## ################################################ ############### ############### ########### ########### ######## ######## ####### ####### ###### ###### ##### ##### ##### ##### ##### ##### #### #### ##### ##### ##### ##### ##### ##### ##### ##### #### ##### ##### #### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### #### #### ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# #### ############# ######### #### ######### ####### #### ####### ###### #### ###### ##### #### ##### ##### #### ##### #### #### #### #### #### #### #### #### #### #### #### #### #### #### ##### ### #### ##### #### #### ###### ### #### ####### #### #### ######### #### #### ############# #### ######################### ##### ######################## ##### ###################### #### #################### ################## ################ ########### #### #### #### #### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### ###################################################### #### ###### #### ###### ####### ####### ####### ####### ######## ######## ######## ############### ################# ################## ################### ################### ################## ################## ################ ############# ######## #### #### #### ######################################################## ############################################################ ############################################################## ################################################################ ################################################################# ################################################################## ################################################################### ################################################################### ########### #### ######### #### ######## #### ####### #### ###### ##### ##### #### ### ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# #### ############# ######### #### ######### ####### #### ####### ###### #### ###### ##### #### ##### ##### #### ##### #### #### #### #### #### #### #### #### #### #### #### #### #### #### ##### ### #### ##### #### #### ###### ### #### ####### #### #### ######### #### #### ############# #### ######################### ##### ######################## ##### ###################### #### #################### ################## ################ ########### ############# ######## ############# ############## ############# ################## ############# ###################### ######## ######################## ####### ########################## ###### ############################ ###### ################ ###### ##### ############## #### ##### ############## #### ##### ############# #### #### ############# #### #### ############ #### #### ############# #### #### ############# #### #### ############# ##### #### ############# ##### ##### ############## ###### #### ############# ###### ##### ############## ####### ##### ############## ######## ###### ################ ############# ########################### ############# ########################## ############# ######################## ############# #################### ################ ######### #### #### #### ######################################################## ############################################################ ############################################################## ################################################################ ################################################################# ################################################################## ################################################################### ################################################################### ########### #### ######### #### ######## #### ####### #### ###### ##### ##### #### ### #### #### #### ######################################################## ############################################################ ############################################################## ################################################################ ################################################################# ################################################################## ################################################################### ################################################################### ########### #### ######### #### ######## #### ####### #### ###### ##### ##### #### ### ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# ############# ######### ######### ####### ####### ###### ###### ##### ##### ##### ##### #### #### #### #### #### #### #### #### ##### ##### ##### ##### ##### ##### ###### ###### ####### ####### ######### ######### ############# ############# ############################################## ############################################ ######################################## #################################### ################################ ############################ ################## ################## ############################ ################################ #################################### ######################################## ############################################ ############################################## ############# ############# ######### ######### ####### ####### ###### ###### ##### ##### ##### ##### #### #### #### #### #### #### #### #### #### #### ### #### #### #### ### #### #### ###### #### ####### #### ######## ##### ####### ##### #### #### # #### #### #### #### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### ########################################################################################### #### ##### #### ##### ##### ##### ###### ###### ###### ####### ####### ######## #### ######### #### ############ ###################################################### ##################################################### ##################################################### #################################################### ################################################### ################################################# ############################################## ########################################## #### #### ################## ############################ ################################ #################################### ######################################## ############################################ ################…

WilcoFiers

I'd like to see this addressed:
#3108 (comment)

Plus, my comment mentions that there are other elements for which this issue exists. This PR should include all of them IMO. As it is, this seems incomplete to me.

resolved

lib/standards/html-elms.js

test/commons/aria/get-element-unallowed-roles.js

lib/commons/aria/get-element-unallowed-roles.js

test/integration/rules/aria-allowed-role/aria-allowed-role.html

lib/commons/aria/get-element-unallowed-roles.js

Resolved

lib/commons/aria/get-element-unallowed-roles.js

…headerrole3108

resolved

lib/commons/aria/is-aria-role-allowed-on-element.js

lib/commons/aria/get-element-unallowed-roles.js

test/commons/aria/get-element-unallowed-roles.js

dylanb

put the <tr> test case back

…headerrole3108

resolved

Added banner role to header element

23a1d92

Closes Issue: #3108

Zidious requested a review from a team as a code owner August 31, 2021 20:36

Zidious changed the title ~~Added banner role to header element~~ feat: added banner role to header element Sep 1, 2021

Zidious requested a review from a team September 1, 2021 15:31

WilcoFiers previously requested changes Sep 17, 2021

View reviewed changes

add check for unallowed-roles on header elm

f4655ad

Zidious requested a review from WilcoFiers October 15, 2021 17:28

add role=contentinfo footer html elm

02d38a1

Zidious changed the title ~~feat: added banner role to header element~~ feat: add role=banner to header and role=contentinfo to footer Oct 15, 2021

WilcoFiers previously requested changes Oct 25, 2021

View reviewed changes

WilcoFiers reviewed Oct 25, 2021

View reviewed changes

test/integration/rules/aria-allowed-role/aria-allowed-role.html Outdated Show resolved Hide resolved

Zidious added 2 commits October 25, 2021 20:55

changes requested

843ef70

changes requested

2b031c5

WilcoFiers reviewed Oct 27, 2021

View reviewed changes

lib/commons/aria/get-element-unallowed-roles.js Outdated Show resolved Hide resolved

Zidious added 3 commits October 27, 2021 17:41

changes requested

ff147d6

clean up dpub if-block

6eb5ef2

add dpub role tests

2caa508

Zidious changed the title ~~feat: add role=banner to header and role=contentinfo to footer~~ fix(aria-allowed-role): landmark roles banner on header and contentinfo on footer to only report on top-level rule Oct 27, 2021

WilcoFiers previously requested changes Oct 28, 2021

View reviewed changes

lib/commons/aria/get-element-unallowed-roles.js Outdated Show resolved Hide resolved

lib/commons/aria/get-element-unallowed-roles.js Outdated Show resolved Hide resolved

Zidious added 4 commits November 2, 2021 13:16

Merge branch 'develop' of https://github.com/dequelabs/axe-core into …

6138c76

…headerrole3108

changes requested

de1480d

Merge branch 'develop' of https://github.com/dequelabs/axe-core into …

ed7d7a7

…headerrole3108

add tests for roles on axe.configure and add tr case

f3406bf

Zidious requested a review from straker November 9, 2021 15:24

changes requested

dd6e828

Zidious commented Nov 10, 2021

View reviewed changes

lib/commons/aria/is-aria-role-allowed-on-element.js Outdated Show resolved Hide resolved

chore(getElementUnallowedRoles): Clean-up work

b1ee30f

WilcoFiers mentioned this pull request Nov 10, 2021

chore(getElementUnallowedRoles): Clean-up work #3271

Merged

WilcoFiers and others added 4 commits November 10, 2021 14:28

chore: minor tweak

e8fe28c

Merge branch 'develop' into headerrole3108

2c8134e

fix integration tests

9988ebd

Merge branch 'develop' into headerrole3108

2df5d81

WilcoFiers requested a review from dylanb November 15, 2021 10:51

WilcoFiers reviewed Nov 15, 2021

View reviewed changes

lib/commons/aria/is-aria-role-allowed-on-element.js Outdated Show resolved Hide resolved

chore(typo): Update lib/commons/aria/is-aria-role-allowed-on-element.js

525f701

dylanb reviewed Nov 17, 2021

View reviewed changes

lib/commons/aria/get-element-unallowed-roles.js Show resolved Hide resolved

dylanb reviewed Nov 17, 2021

View reviewed changes

lib/commons/aria/get-element-unallowed-roles.js Show resolved Hide resolved

dylanb reviewed Nov 17, 2021

View reviewed changes

test/commons/aria/get-element-unallowed-roles.js Show resolved Hide resolved

Zidious added 2 commits November 17, 2021 18:01

fix merge conflicts

2bec405

fix merge conflicts

00dbbc1

dylanb previously requested changes Nov 18, 2021

View reviewed changes

Zidious added 5 commits November 18, 2021 15:15

add TR test case back

459eead

Merge branch 'develop' of https://github.com/dequelabs/axe-core into …

0ea15d3

…headerrole3108

merge changes

ec6047f

remove duplicate integration tests

9170f27

Merge branch 'develop' of https://github.com/dequelabs/axe-core into …

d8f858b

…headerrole3108

dylanb approved these changes Nov 24, 2021

View reviewed changes

Zidious merged commit 1fd4b00 into develop Nov 24, 2021

Zidious deleted the headerrole3108 branch November 24, 2021 13:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(aria-allowed-role): landmark roles banner on header and contentinfo on footer to only report on top-level rule#3142

fix(aria-allowed-role): landmark roles banner on header and contentinfo on footer to only report on top-level rule#3142
Zidious merged 27 commits intodevelopfrom
headerrole3108

Zidious commented Aug 31, 2021 •

edited

Loading

Uh oh!

WilcoFiers left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dylanb left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Zidious commented Aug 31, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

WilcoFiers left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dylanb left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Zidious commented Aug 31, 2021 •

edited

Loading