- Issue created by @simohell
- ๐ซ๐ฎFinland simohell
An example from Voice over rotor why the use of article-tags for each media can be a problem.
- ๐ฎ๐ณIndia Gauravvv Delhi, India
Gauravvvv โ made their first commit to this issueโs fork.
- Merge request !7598Article tag replaced with div in media.html.twig file โ (Open) created by Unnamed author
- Status changed to Needs review
5 months ago 3:24am 19 April 2024 - Status changed to Active
5 months ago 6:32pm 19 April 2024 - ๐บ๐ธUnited States bnjmnm Ann Arbor, MI
It looks like the
<article>
use extends beyond Claro - it's in additional places such as the core module template and Stable 9. This could be disruptive to existing sites using<article>
as a selector for styling/JS/tests so it really shouldn't happen in 10 outside of Starterkit. Thi change could be introduced in 11, though!The adverse effects of this excess
<article>
use can be somewhat mitigated by labeling these instances usingaria-label
or a similar approach. That way, assistive tech would see something more descriptive than just ArticleI'm going to keep the "Needs subsystem maintainer review" around as I suspect an additional review will be needed, but that's my review for this stage of the process โ๏ธ