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

Table column resizing aria docs #3982

Merged
merged 94 commits into from
Apr 5, 2023
Merged

Table column resizing aria docs #3982

merged 94 commits into from
Apr 5, 2023

Conversation

LFDanLu
Copy link
Member

@LFDanLu LFDanLu commented Jan 31, 2023

Closes

✅ 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:

Looks at the column resizing docs for useTableState and useTable. In addition, try the styled example out here: https://codesandbox.io/s/objective-cherry-g837on?file=/src/App.tsx

🧢 Your Project:

RSP

@rspbot
Copy link

rspbot commented Jan 31, 2023

@rspbot
Copy link

rspbot commented Jan 31, 2023

@rspbot
Copy link

rspbot commented Jan 31, 2023

@rspbot

This comment was marked as outdated.

@LFDanLu
Copy link
Member Author

LFDanLu commented Mar 31, 2023

TODO: update sandbox example so it uses input styles from useTableColumnResize hook instead of VisuallyHidden, need to wait for nightly/release for that

@rspbot
Copy link

rspbot commented Mar 31, 2023

@rspbot
Copy link

rspbot commented Mar 31, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

snowystinger
snowystinger previously approved these changes Apr 5, 2023
@LFDanLu LFDanLu changed the title (DNM) Table column resizing aria docs Table column resizing aria docs Apr 5, 2023
@rspbot
Copy link

rspbot commented Apr 5, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

@rspbot
Copy link

rspbot commented Apr 5, 2023

## API Changes

unknown top level export { type: 'identifier', name: 'Column' }
unknown top level export { type: 'identifier', name: 'Column' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }

@LFDanLu LFDanLu merged commit b06a446 into main Apr 5, 2023
@LFDanLu LFDanLu deleted the aria_table_docs branch April 5, 2023 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants