[x] Remove “Uses Astro for […]” from features and merge it with the lead “Front-end Astro boilerplate […]“. Astro is not a feature as much as it’s the « raison d’être » of the boilerplate. It’s not something you can swap out with another framework.
[x] Is the “front-end” keyword necessary? Isn’t Astro all-in-one / SSG where your data baked in directly without a “back-end”.
[x] Under “Features”, remove “Uses”, it’s redundant.
[x] Move “Development” below “Project Structure” to be closer to “Commands”.
Accordion:
[x] Misused/misunderstood and unnecessary aria-expanded attribute on the <details>element.
[x] The aria-expanded attribute must be applied to an interactive control (i.e., a button). For the disclosure element, the interactive control is the <summary>.
[x] According to these outdated accessibility tests: The aria-expanded attribute could be practical on for Narrator on Edge, NVDA on Chrome and Edge, and VoiceOver on iOS. Do NOT add the attribute for the time being.
[x] There should be support and documentation for the name attribute for creating exclusive accordions.
Blocks
[x] Missing documentation.
Button
[x] According to the logic, the target attribute is not rendered.
[x] There should be a computed rel, target, and aria-hidden to ensure they are not accidentally rendered.
[ ] The default value of isStatic is 'div' instead of false.
[x] The ↗ (<span>↗</span>) needs to be either hidden or have a proper accessible name. Otherwise, AT’s will read “North East Arrow” after the main label.
Icon
[x] Is xlink:href still necessary; could it be just href?
[x] A label option could be added to support non-decorative icons (if label attribute present, replaces aria-hidden and focusable.
[x] Replace “True” with “Yes” for consistency.
Wysiwyg
[x] Expand selection of acceptable tags. div and article is narrow and insufficient. Should include main, section, aside, other sectioning tags.
[x] Replace « Non » with “No” for consistency.
[x] Replace “‘div’” with 'div' (as code) for consistency.
Accordion:
<details>
element.<summary>
.Blocks
Button
<span>↗</span>
) needs to be either hidden or have a proper accessible name. Otherwise, AT’s will read “North East Arrow” after the main label.Icon
Wysiwyg