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.

Sign up for GitHub

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

Jump to bottom

[BITV] 9.1.3.1b/8.2 - The list of workdays (see screenshot) is currently not structured, meaning it has not been implemented as a list, table, or with the use of headings. It is recommended to structure the workdays as a list to make the content more accessible. (1) #190

Closed
JuliaKirschenheuter opened this issue Nov 29, 2023 · 5 comments · Fixed by #191
Closed

[BITV] 9.1.3.1b/8.2 - The list of workdays (see screenshot) is currently not structured, meaning it has not been implemented as a list, table, or with the use of headings. It is recommended to structure the workdays as a list to make the content more accessible. (1) #190

JuliaKirschenheuter opened this issue Nov 29, 2023 · 5 comments · Fixed by #191
Assignees
@emoral435
Labels
a11y28checked

Comments

@JuliaKirschenheuter
Copy link
Contributor

JuliaKirschenheuter commented Nov 29, 2023 •
edited
Loading

Additionally, it would be advisable to give the list a name, for example, using the "aria-labelledby" attribute.

`Workdays

    ...
`

image

https://report.bitvtest.de/default-en/cbedb5ef-4815-4cba-a83d-995599c178c3.html#checkpoint-1d5eadb58d-v8-n2
@paramazo

This comment was marked as off-topic.

Sign in to view
@JuliaKirschenheuter JuliaKirschenheuter changed the title [BITV] 9.1.3.1b/8.2 - The list of workdays (see screenshot) is currently not structured, meaning it has not been implemented as a list, table, or with the use of headings. It is recommended to structure the workdays as a list to make the content more accessible. Additionally, it would be advisable to give the list a name, for example, using the "aria-labelledby" attribute. (1) [BITV] 9.1.3.1b/8.2 - The list of workdays (see screenshot) is currently not structured, meaning it has not been implemented as a list, table, or with the use of headings. It is recommended to structure the workdays as a list to make the content more accessible. (1) Nov 29, 2023
@susnux
Copy link
Contributor

susnux commented Nov 30, 2023

it is a calender component?

No this is the availability settings which is part of this repository and thus it is correctly placed in this bug tracker.

@emoral435 emoral435 self-assigned this Dec 3, 2023
@emoral435 emoral435 transferred this issue from nextcloud/server Dec 4, 2023
@emoral435 emoral435 mentioned this issue Dec 5, 2023
Merged
@szaimen
Copy link

szaimen commented Jan 5, 2024

This is not here yet, I guess an update of the library in stable28 is missing

@szaimen szaimen reopened this Jan 5, 2024
@emoral435 emoral435 mentioned this issue Jan 5, 2024
Merged
@emoral435
Copy link
Contributor

emoral435 commented Jan 8, 2024 •
edited
Loading

Will be fixed once nextcloud/server#42629 gets merged 🙂!

@emoral435
Copy link
Contributor

Merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emoral435 emoral435

Labels
a11y28checked
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(aria) - changed workdays to list format nextcloud/calendar-availability-vue
5 participants
@susnux @JuliaKirschenheuter @szaimen @emoral435 @paramazo