-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[accessibility] Remove role="list" from Attribution #12857
Conversation
@@ -64,7 +64,6 @@ class AttributionControl { | |||
this._compactButton.addEventListener('click', this._toggleAttribution); | |||
this._setElementTitle(this._compactButton, 'ToggleAttribution'); | |||
this._innerContainer = DOM.create('div', 'mapboxgl-ctrl-attrib-inner', this._container); | |||
this._innerContainer.setAttribute('role', 'list'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kumiko-haraguchi could we just include this change in your PR and revert those other changes? I can make the upstream changes to remove listitem
without having to intercept them in the dom.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Followup to remove listitem
from TileJSON records will happen upstream.
73706e6
to
c5e1778
Compare
Handling the good first issue #11033, removing
role="list"
androle="listitem"
from Attribution.[Before]
[After]
Note about removing role="listitem"
Attribution link elements seem to be provided from TileJSON and this original source should be fixed.
I don't seem to have access to fix it so I removed it from the attribution source obtained bythis._map.style._sourceCaches
in this PR as a quick fix.I hope someone in the Mapbox team could handle this part.
role="listitem"
from TileJSONLaunch Checklist
mapbox-gl-js
changelog:<changelog>Remove role="list" and role="listitem" from Attribution</changelog>