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(slider): add new component #3378
Conversation
Run & review this pull request in StackBlitz Codeflow. |
🦋 Changeset detectedLatest commit: bea95d6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
7be3474
to
6b13270
Compare
✅ Deploy Preview for paste-theme-designer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit bea95d6. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
✅ Deploy Preview for paste-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
6b13270
to
4057b25
Compare
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 6b13270:
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit bea95d6:
|
Size Change: +238 B (0%) Total Size: 1.02 MB
ℹ️ View Unchanged
|
1 failed tests on run #6485 ↗︎
Details:
cypress/integration/sitemap-vrt/batch2.spec.ts • 1 failed test • Paste Actions Parallel
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
/** | ||
* Used to adjust how the numbers are rendered and interpreted. | ||
* https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat \ | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
praise: useful TSDoc comments
packages/paste-core/components/slider/stories/index.stories.tsx
Outdated
Show resolved
Hide resolved
955a3dd
to
3334e66
Compare
}); | ||
}); | ||
|
||
describe('Customization', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: tests for applying custom styles?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those are handled by Chromatic. I realize we duplicate a lot of effort everywhere, figured I'd break the trend.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh nice
3334e66
to
1811656
Compare
Adds the new Slider package based on React-Spectrum's headless UI hooks.
Spec document