Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

Update Calendar markup for a11y #559

Open
jacobdevera opened this issue Nov 1, 2019 · 1 comment
Open

Update Calendar markup for a11y #559

jacobdevera opened this issue Nov 1, 2019 · 1 comment

Comments

@jacobdevera
Copy link

jacobdevera commented Nov 1, 2019

Is this a bug, enhancement, or feature request?

Enhancement (to comply with a11y)

Briefly describe your proposal.

The rendered markup should be updated to reflect changes made in fundamental-styles.
SAP/fundamental-styles#410

  • Convert month/year selector ul into table with role=“grid”
  • Currently selected date: <td>: aria-selected=“true” and <span> role "button"

Additional changes:

  • Month/year header: aria-live=“polite” (to announce month/year changes)
  • Navigation buttons: Needs localizable aria-label (e.g. “Next Year”, "Next Month")
  • Selectable dates: aria-label should be used to read out full date
  • Disabled dates: <td>: aria-disabled="true"
@ChristianKienle
Copy link
Contributor

this will be fixed with the next release. i have already implemented it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants