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

[css-view-transitions-1] Include aria-busy="true" by default #8447

Closed
nattarnoff opened this issue Feb 13, 2023 · 5 comments
Closed

[css-view-transitions-1] Include aria-busy="true" by default #8447

nattarnoff opened this issue Feb 13, 2023 · 5 comments
Assignees
Labels
css-view-transitions-1 View Transitions; Bugs only

Comments

@nattarnoff
Copy link

[css-view-transitions-1]
We've had aria-busy for a little why now to indicate when dynamically loaded content is being fetched and rendered to the client. This is something the author needs to set though. With CSS View Transitions there will be a significant amount of content updating, moving, and changing. Users who are blind or have low vision and are reliant on screen readers will need to know that the content is in a change state. I'm proposing that instead of leaving it up to the author to include aria-busy="true" when they are developing the transitions the specification should include this during the transition. This will improve accessibility and reduce the weight to implement the CSS feature by reassuring basic accessibility is covered.

@jakearchibald
Copy link
Contributor

Wouldn't this be more appropriate for the navigation API? Transitions should come after content fetching, so it seems like a busy period should start before view transitions are involved, and from (for example) a screen reader's point of view, the content may be fully available from the start of the transition.

@khushalsagar
Copy link
Member

+1 to Jake's feedback, the idea with ViewTransitions is for authors to call the API after fetching content from the network. Since as soon as you call document.startViewTransition, the browser freezes rendering to let you asynchronously update the DOM.

While its possible that you can start the transition while some parts of your page are still loading, the transition API is agnostic to that and can't tell whether the DOM is fully loaded. So it doesn't make sense for aria-busy to be set automatically.

@khushalsagar khushalsagar added the css-view-transitions-1 View Transitions; Bugs only label May 11, 2023
@noamr
Copy link
Collaborator

noamr commented May 19, 2023

Is this actionable?

@khushalsagar
Copy link
Member

Let's give nattarnoff another week to reply before closing.

@nattarnoff
Copy link
Author

Closing. I haven't had time to play and understand the concept deeper than what is laid out here. If it is an issue in the future, I'll open another issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-view-transitions-1 View Transitions; Bugs only
Projects
None yet
Development

No branches or pull requests

4 participants