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

Docs/storybook locale control #38

Merged
merged 2 commits into from May 24, 2022

Conversation

tomchentw
Copy link
Contributor

This PR adds the locale control to the Storybook CustomLocale component. It also upgrades storybook dependencies for the reason below in Notes section. Please feel free to drop the first commit though.

Screenshots

JPEG-2

d815237ca7148143270ddaf0bb2a5e8d

Notes

Not sure why, but I have to update the highlighted code at L385 on my machine in order to run Storybook. I tried upgrading Storybook dependencies but it seems no luck.

JPEG-1

@netlify
Copy link

netlify bot commented May 23, 2022

Deploy Preview for uselessdev-datepicker ready!

Name Link
🔨 Latest commit 705162c
🔍 Latest deploy log https://app.netlify.com/sites/uselessdev-datepicker/deploys/628b194c05e2530007bc9683
😎 Deploy Preview https://deploy-preview-38--uselessdev-datepicker.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 settings.

@uselessdev
Copy link
Owner

Thanks @tomchentw this looks really nice

@uselessdev uselessdev merged commit 1c1c327 into uselessdev:main May 24, 2022
@uselessdev
Copy link
Owner

@all-contributors please add @tomchentw for docs and code

@allcontributors
Copy link
Contributor

@uselessdev

I've put up a pull request to add @tomchentw! 🎉

@tomchentw tomchentw deleted the docs/storybook-locale-control branch May 25, 2022 03:08
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.

None yet

2 participants