Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

a11y: add announcements to adaptive-form#2722

Closed
beyackle wants to merge 2 commits intomasterfrom
beyackle/announceNewItem
Closed

a11y: add announcements to adaptive-form#2722
beyackle wants to merge 2 commits intomasterfrom
beyackle/announceNewItem

Conversation

@beyackle
Copy link
Contributor

Description

This wires through an "announce" prop all the way into adaptive-form so we can take advantage of the global announcement region from within the form editor. This change was needed to solve an a11y issue, but should be useful for other tasks as well.

Task Item

Closes #2130

@github-actions
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 351a8be on beyackle/announceNewItem into 2936a71 on master.

@beyackle
Copy link
Contributor Author

closing in favor of #2734, which solves the problem in a much cleaner way

@beyackle beyackle closed this Apr 21, 2020
@beyackle beyackle deleted the beyackle/announceNewItem branch April 21, 2020 20:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant