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

UI/ember a11y refocus #2837

Merged
merged 3 commits into from
Jan 5, 2022
Merged

UI/ember a11y refocus #2837

merged 3 commits into from
Jan 5, 2022

Conversation

sabrinako
Copy link
Contributor

Note: should there be a changelog for this?

Context

  • This is work from horde programming time to implement ember-a11y-refocus
  • Currently, there's still an issue where every time polling occurs, the transition element is refocused on
    • This will be addressed in a future PR (see branch: ui/ember-a11y-refocus-model-refactor)

@sabrinako sabrinako added ui ui/a11y an accessibility issue labels Dec 15, 2021
@sabrinako sabrinako requested a review from a team December 15, 2021 16:51
@github-actions
Copy link

github-actions bot commented Dec 15, 2021

Ember Asset Size action

As of 6149cb9

Files that got Bigger 🚨:

File raw gzip
vendor.js +4.99 kB +1.05 kB
waypoint.js +15.7 kB +3.27 kB
vendor.css +427 B +110 B
waypoint.css +762 B +299 B

@gregone
Copy link
Contributor

gregone commented Jan 4, 2022

@sabrinako I think we could add a changelog entry on this and merge. How do you feel about that?

@sabrinako
Copy link
Contributor Author

@sabrinako I think we could add a changelog entry on this and merge. How do you feel about that?

@gregone sounds good to me! I'll do that today

@sabrinako sabrinako linked an issue Jan 5, 2022 that may be closed by this pull request
@sabrinako sabrinako merged commit fa724f2 into main Jan 5, 2022
@sabrinako sabrinako deleted the ui/ember-a11y-refocus branch January 5, 2022 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ui/a11y an accessibility issue ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use ember-a11y-refocus
2 participants