Skip to content
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

[Bug Report][2.6.2] v-calendar should use table elements in markup #14604

Closed
apfrod opened this issue Jan 13, 2022 · 1 comment · Fixed by #14640
Closed

[Bug Report][2.6.2] v-calendar should use table elements in markup #14604

apfrod opened this issue Jan 13, 2022 · 1 comment · Fixed by #14640
Labels
a11y Accessibility issue C: VCalendar VCalendar T: enhancement Functionality that enhances existing features
Milestone

Comments

@apfrod
Copy link

apfrod commented Jan 13, 2022

Environment

Vuetify Version: 2.6.2
Vue Version: 2.6.14
Browsers: Chrome 97.0.4692.71
OS: Mac OS 10.15.7

Steps to reproduce

I have received the following report from an accessibility tester regarding v-calendar:
The table content such as "SUN", "MON", "TUE" etc. is presented using <div> elements.
As a result, screen reader users cannot understand the data table content effectively.

Expected Behavior

Apply the following changes:

  • Utilized table mark-up such as <table>, <tr>, <th>, <td> etc. to present the content as a data table.
  • Mark-up "SUN", "MON", "TUE" etc. table headers using <th> element.

Actual Behavior

The table content such as "SUN", "MON", "TUE" etc. is presented using <div> elements.

Reproduction Link

https://codepen.io/apfrod/pen/WNZLRrb

@KaelWD KaelWD added a11y Accessibility issue C: VCalendar VCalendar T: enhancement Functionality that enhances existing features and removed S: triage labels Jan 19, 2022
@KaelWD
Copy link
Member

KaelWD commented Jan 19, 2022

This should just use ARIA roles instead, actual tables have some pretty annoying layout implications.

knknk98 added a commit to knknk98/vuetify that referenced this issue Jan 24, 2022
@KaelWD KaelWD added this to the v2.6.x milestone Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility issue C: VCalendar VCalendar T: enhancement Functionality that enhances existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants