react icon indicating copy to clipboard operation
react copied to clipboard

Accessibility fixes for ToggleSwitch

Open TylerJDev opened this issue 1 year ago • 14 comments

Closes https://github.com/github/primer/issues/3463

Stemming from @camertron's work in https://github.com/primer/react/pull/3902

Example story

Changelog

New

  • Adds new props loadingLabelDelay and loadingLabel

Changed

  • Makes aria-labelledby a required prop
  • Utilizes aria-disabled instead of disabled

Rollout strategy

  • [ ] Patch release
  • [x] Minor release
  • [ ] Major release; if selected, include a written rollout or migration plan
  • [ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

TylerJDev avatar Jul 16 '24 20:07 TylerJDev

🦋 Changeset detected

Latest commit: 3d240415fa66f7c55f76a1565a07009c53d00306

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/react Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Jul 16 '24 20:07 changeset-bot[bot]

size-limit report 📦

Path Size
packages/react/dist/browser.esm.js 90.23 KB (+0.23% 🔺)
packages/react/dist/browser.umd.js 90.47 KB (+0.39% 🔺)

github-actions[bot] avatar Jul 16 '24 21:07 github-actions[bot]

@joshblack - if you get the chance, I'd love a review from you on this PR, mainly because I added onto Announce.tsx, and I want to make sure it's the right change :grin:

TylerJDev avatar Aug 20 '24 14:08 TylerJDev

Heads up @tylerjdev, Josh is out of office for a bit (check our team calendar for dates). Feel free to leave it until then, but wanted to let you know in case it made sense to request a different reviewer.

lesliecdubs avatar Aug 26 '24 19:08 lesliecdubs

:wave: Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks!

github-actions[bot] avatar Oct 24 '24 22:10 github-actions[bot]

Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

github-actions[bot] avatar Jan 14 '25 17:01 github-actions[bot]

👋🏻 @TylerJDev should we revisit this one to get it merged in?

lesliecdubs avatar Jan 14 '25 21:01 lesliecdubs

@lesliecdubs, definitely! Will put this on my to-do for next week to ship! 🚢

TylerJDev avatar Jan 17 '25 14:01 TylerJDev

Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

github-actions[bot] avatar Mar 18 '25 15:03 github-actions[bot]

To be unblocked by https://github.com/github/github/pull/371043. Once merged, this PR should be good to ship!

TylerJDev avatar Apr 02 '25 23:04 TylerJDev

Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

github-actions[bot] avatar Jun 02 '25 00:06 github-actions[bot]

@joshblack, mind giving this one another quick review? Shouldn't include any additional changes since the last approval, but it's been a bit since then 😅

TylerJDev avatar Jun 03 '25 20:06 TylerJDev

Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

github-actions[bot] avatar Aug 03 '25 20:08 github-actions[bot]

:wave: Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

github-actions[bot] avatar Aug 04 '25 14:08 github-actions[bot]

:wave: Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

github-actions[bot] avatar Aug 04 '25 15:08 github-actions[bot]

:wave: Hi from github/github! Your integration PR is ready: https://github.com/github/github/pull/394584

primer-integration[bot] avatar Aug 04 '25 15:08 primer-integration[bot]

🟢 golden-jobs completed with status success.

primer-integration[bot] avatar Aug 04 '25 16:08 primer-integration[bot]

:wave: Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

github-actions[bot] avatar Aug 04 '25 19:08 github-actions[bot]

:wave: Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

github-actions[bot] avatar Aug 04 '25 19:08 github-actions[bot]