Closed (won't fix)
Project:
Drupal core
Version:
9.3.x-dev
Component:
Olivero theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Aug 2021 at 07:25 UTC
Updated:
9 Aug 2021 at 15:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
gauravvvv commentedI have provided a patch with aria role added to footer element. Please review.
Ref document for Aria roles in HTML5 https://dequeuniversity.com/assets/html/jquery-summit/html5/slides/landm...
Comment #3
vikashsoni commented@Gauravmahlawat thanks for the patch
successfully added Add role for footer element
for ref sharing screenshot
Comment #4
bnjmnm@vikashsoni there is no benefit in showing the diff between a patch and HEAD, particularly as that is how the patch is generated to begin with. It's is essentially just taking a screenshot of the patch (not the results of the patch, literally this patch) in #2. Removing credit as per How is credit granted for Drupal core issues.
Comment #5
radheymkumar commented@Gauravmahlawat thanks for the patch
Comment #6
bnjmnmRe #5 @radheymkumar
I noticed you posted a diff of the patch. In the previous comment I'd just said such a comment provides no benefit nor will it earn issue credit.
I ALSO have to point out it is the exact same image as the (not useful )screenshot provided in #3, with a different filename.
MD5 doesn't lie 🙂

Comment #7
mherchel@radheymkumar I would love you hear your explanation for #6
Comment #8
mherchelI'd love to get an accessibility maintainer's thoughts on this. I know the "first rule of aria" is not to use aria, so I'm not going to OK this unless we have a real reason to do so.
Comment #9
larowlanAgree, this feels superfluous
Comment #10
bnjmnmI got so distracted by the dupe screenshot I neglected to help with accessibility.
This is currently needed, but not for long 🤔. This is needed by Safari 12, which is still supported as part of Drupal's supported browsers policy, where Safari is among the browsers Drupal supports "The latest release of each of the latest two supported major versions of". Safari 12 will not longer be supported by Apple as of November 30, 2021, however. Seems excessive, but may be worth accompanying this change with a
@todo,Or if the issue gets bikeshedded for ~3 months you can just close it as outdated 🤓☠️.
Comment #11
mherchelYeah, the next release isn't coming out until December, so my thought is that there's no point in making this change.