gsoft-inc / wl-hopper

Hopper Design System
https://hopper.workleap.design/
Apache License 2.0
3 stars 2 forks source link

Reorganized grouping of props to follow RAC #534

Closed fraincs closed 4 days ago

fraincs commented 4 days ago

Grouping of props in the documentation props table was flaky, in some instances isDisabled was considered an accessibility prop, where in other place it was not. This PR follows RAC ways of grouping props, while making sure than an id? string is considered an accessibility prop and an id? key is considered exposed in the "default" props.

changeset-bot[bot] commented 4 days ago

⚠️ No Changeset found

Latest commit: ce719b4c0f74d0c07e182ee5ad205e5e56149049

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

netlify[bot] commented 4 days ago

Deploy Preview for wl-hopper ready!

Name Link
Latest commit ce719b4c0f74d0c07e182ee5ad205e5e56149049
Latest deploy log https://app.netlify.com/sites/wl-hopper/deploys/674882f776aec100073fa6fc
Deploy Preview https://deploy-preview-534--wl-hopper.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 4 days ago

View Storybook