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

Create Storybook stories for Penumbra colors #1500

Merged
merged 1 commit into from
Jul 21, 2024

Conversation

jessepinho
Copy link
Contributor

@jessepinho jessepinho commented Jul 17, 2024

Creates this page in Storybook

Copy link

changeset-bot bot commented Jul 17, 2024

⚠️ No Changeset found

Latest commit: ae32dbb

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

@jessepinho jessepinho changed the base branch from jessepinho/sc-theme to jessepinho/new-components-1 July 17, 2024 21:35
Copy link
Contributor

Visit the preview URL for this PR (updated for commit ae32dbb):

https://penumbra-ui-preview--pr1500-jessepinho-colors-xw8bzgdf.web.app

(expires Wed, 24 Jul 2024 21:36:54 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 709d729610ef7a6369b23f1cb2b820a60cc685b1

@jessepinho jessepinho marked this pull request as ready for review July 17, 2024 21:40
@jessepinho jessepinho requested a review from a team July 17, 2024 21:44
@@ -0,0 +1,108 @@
import type { Meta, StoryObj } from '@storybook/react';
import { Grid } from './Grid';
import { Technical } from './Typography';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: are these changes coming from #1489? It confused me since Technical and other components were not reviewed yet but the questions are mostly about them, not the color stories 😅

It's just that even if we merge this PR now, these changes will need to be reviewed once again with other changes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@VanishMax ah, yes, but I haven't made that PR ready for review yet (oops). My plan was to wait until they're all approved, then merge them into each other and finally into main. If you give this the 👍🏻 , I'll wait to merge it until #1489 is approved and merged.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just marked #1489 ready for review as well

@TalDerei TalDerei self-requested a review July 20, 2024 04:23
@jessepinho jessepinho merged commit a29919f into jessepinho/new-components-1 Jul 21, 2024
8 checks passed
@jessepinho jessepinho deleted the jessepinho/colors branch July 21, 2024 20:58
jessepinho added a commit that referenced this pull request Jul 21, 2024
* Build new components

* Fix quotation marks

* Move colors to other PR

* Delete button-related components for now

* Split out typography components

* Add docs for Detail

* Document typography file

* Fix import

* Fix a few issues with how Storybook displays the Grid story

* Revert "Split out typography components"

This reverts commit 7740d3f.

* Create a Text component

* Create Storybook stories for Penumbra colors (#1500)
jessepinho added a commit that referenced this pull request Jul 21, 2024
* Build out styled-components theme

* Add Iosevka Term; add quotation marks

* Add comment

* Add comment

* Build out new UI components (#1489)

* Build new components

* Fix quotation marks

* Move colors to other PR

* Delete button-related components for now

* Split out typography components

* Add docs for Detail

* Document typography file

* Fix import

* Fix a few issues with how Storybook displays the Grid story

* Revert "Split out typography components"

This reverts commit 7740d3f.

* Create a Text component

* Create Storybook stories for Penumbra colors (#1500)

* Fix imports
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.

3 participants