Skip to content

Cleaning up storybook files#5290

Merged
emrysal merged 40 commits intomainfrom
storybook-cleanup
Nov 4, 2022
Merged

Cleaning up storybook files#5290
emrysal merged 40 commits intomainfrom
storybook-cleanup

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

@sean-brydon sean-brydon commented Oct 31, 2022

Hello!

With help of @JeroenReumkens we have been working on creating proper design documentation to match our Figma as close as possible. During this time we are also starting to migrate away from V2 folder into a @calcom/ui/components This will be the final directory of all UI components. Once they have undergone review - and also had the relevant documentation created for them.

As you can see we have only migrated 5 components in this PR and there are a lot of file changes. So we're gonna push for this bulk to get reviewed and merged. From here we will work on doing it as a simple component at the time to keep the amount of changes to the code base to a minimum.

We dropped a readme in the ui folder to explain what's going on in some more detail.
CleanShot 2022-11-02 at 13 26 47

https://www.loom.com/share/df708fdac46b470ea201ef632b1f30a6

TODO:

  • Update website, console, api with new paths
  • Delete v1 versions of new components

These need to be promoted to production when this PR goes live.
Website: https://github.com/calcom/website/pull/393
Console: https://github.com/calcom/console/pull/39

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ❌ Failed (Inspect) Nov 4, 2022 at 3:29PM (UTC)

@sean-brydon sean-brydon marked this pull request as ready for review November 4, 2022 12:41
@sean-brydon sean-brydon changed the title WIP Cleaning up storybook files Cleaning up storybook files Nov 4, 2022
Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Had a once over console and website too and tested through various points of this PR, approving it, but it's very big. Smaller more contained PRs 👍

@sean-brydon
Copy link
Copy Markdown
Member Author

Had a once over console and website too and tested through various points of this PR, approving it, but it's very big. Smaller more contained PRs 👍

One component at a time from now :) I promise

@emrysal emrysal merged commit b5b41da into main Nov 4, 2022
@emrysal emrysal deleted the storybook-cleanup branch November 4, 2022 15:40
emrysal added a commit that referenced this pull request Nov 10, 2022
* storybook v2 init

* Merge config into storybook vite build

* Remove path

* Storybook config tweaks

* Added styles and settings for storybook v2, and started working on button documentation and examples.

* Badges + flex wrap on mobile

* Breadcrumbs+button+avatar

* Checkbox

* Input + moving files around

* WIP table

* WIP table grid

* Replaced imports for new components.

* Added first steps for varianttable.

* Small alignment fix.

* Custom Args Table - With scrollbar

* Adding table to components that need it + darkmode

* Add intro

* Fix types

* Remove V1 storybook and replace with V2

* Fix badge type error

* Fixed storybook dependencies

* Added cover image to storybook

* Remove vita from ts config, we dont use vite.

* Fixed button import.

* Explained postcss pseudo plugin.

* Fixed badge import.

* Add Avatar Stories

* ButtonGroup Stories

* Fixed imports

* Add checkbox stories

* Add  exports for differnt types of inputs

* Fix form and text area input

* Fix mass import errors

Co-authored-by: Jeroen Reumkens <hello@jeroenreumkens.nl>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
emrysal added a commit that referenced this pull request Nov 10, 2022
* storybook v2 init

* Merge config into storybook vite build

* Remove path

* Storybook config tweaks

* Added styles and settings for storybook v2, and started working on button documentation and examples.

* Badges + flex wrap on mobile

* Breadcrumbs+button+avatar

* Checkbox

* Input + moving files around

* WIP table

* WIP table grid

* Replaced imports for new components.

* Added first steps for varianttable.

* Small alignment fix.

* Custom Args Table - With scrollbar

* Adding table to components that need it + darkmode

* Add intro

* Fix types

* Remove V1 storybook and replace with V2

* Fix badge type error

* Fixed storybook dependencies

* Added cover image to storybook

* Remove vita from ts config, we dont use vite.

* Fixed button import.

* Explained postcss pseudo plugin.

* Fixed badge import.

* Add Avatar Stories

* ButtonGroup Stories

* Fixed imports

* Add checkbox stories

* Add  exports for differnt types of inputs

* Fix form and text area input

* Fix mass import errors

Co-authored-by: Jeroen Reumkens <hello@jeroenreumkens.nl>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
haffla pushed a commit to tourlane/cal.com that referenced this pull request Nov 22, 2022
* storybook v2 init

* Merge config into storybook vite build

* Remove path

* Storybook config tweaks

* Added styles and settings for storybook v2, and started working on button documentation and examples.

* Badges + flex wrap on mobile

* Breadcrumbs+button+avatar

* Checkbox

* Input + moving files around

* WIP table

* WIP table grid

* Replaced imports for new components.

* Added first steps for varianttable.

* Small alignment fix.

* Custom Args Table - With scrollbar

* Adding table to components that need it + darkmode

* Add intro

* Fix types

* Remove V1 storybook and replace with V2

* Fix badge type error

* Fixed storybook dependencies

* Added cover image to storybook

* Remove vita from ts config, we dont use vite.

* Fixed button import.

* Explained postcss pseudo plugin.

* Fixed badge import.

* Add Avatar Stories

* ButtonGroup Stories

* Fixed imports

* Add checkbox stories

* Add  exports for differnt types of inputs

* Fix form and text area input

* Fix mass import errors

Co-authored-by: Jeroen Reumkens <hello@jeroenreumkens.nl>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
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.

4 participants