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

feat: 🎸 add press events to Tab #6817

Open
wants to merge 229 commits into
base: main
Choose a base branch
from

Conversation

ArrayKnight
Copy link

Closes: #6810

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

  • yarn start
  • Navigate to Tabs Example in React Aria Components in Storybook
  • Notice that clicking on a Tab fires press event actions
  • Notice no breaking changes to existing tab behavior

Reid Barber and others added 28 commits July 9, 2024 09:46
* fix lint

* add skipLibCheck

* remove skipLibCheck

---------

Co-authored-by: Reid Barber <[email protected]>
* initialize alert dialog

* add alert dialog

* update story, fix closing

* remove comments

* remove props.styles

* update types

* remove orientation prop

* add aria-label to icons

* fix lint

---------

Co-authored-by: Yihui Liao <[email protected]>
* fix rendering TagGroup inside GridList

* clear in GridListItem instead
* Keep focus inside of ComboboxInput
* add script to test docs for warnings/errors

* get links before testing

* update yarn.lock

* update script to get latest browser binaries first

* support running with webkit and firefox

* exit process at the end

---------

Co-authored-by: Robert Snow <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add PressEvents to Tab