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
chore(icons): add theme and color picker icons #1679
Conversation
🦋 Changeset detectedLatest commit: 7d5b87c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
Size Change: 0 B Total Size: 633 kB ℹ️ View Unchanged
|
✔️ Deploy Preview for paste-docs ready! 🔨 Explore the source changes: 7d5b87c 🔍 Inspect the deploy log: https://app.netlify.com/sites/paste-docs/deploys/60f704dd2d297f00070a1690 😎 Browse the preview: https://deploy-preview-1679--paste-docs.netlify.app |
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
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 7d5b87c:
|
.changeset/moody-lies-fry.md
Outdated
@@ -0,0 +1,6 @@ | |||
--- | |||
'@twilio-paste/icons': patch | |||
'@twilio-paste/core': patch |
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.
We don't patch core for icon changes. Icons is the only thing that doesn't also update core.
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.
ah, good to know! Thanks!
@@ -66,14 +66,14 @@ const HomeHero: React.FC = () => { | |||
<Column span={5}> | |||
<NewComponentBanner> | |||
<NewComponentBannerBadge>New!</NewComponentBannerBadge> | |||
<NewComponentBannerText>We just added the Date Picker!</NewComponentBannerText> | |||
<NewComponentBannerText>Did you know we've got an Alert Dialog?!</NewComponentBannerText> |
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.
You may need to rebase now that you merged this change in another PR. I don't think you intended to make this change in this PR
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.
Yeah, I mentioned it at the top, twas a mistake- this change is merged in now so hopefully it won't be too much of a problem
b70dcd4
to
7d5b87c
Compare
Added new icons: Theme and Color Picker
(branched this off of website/new-banner by accident 😬, will remedy when that branch is merged in to main)