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

Add Storybook to the Frontend library #611

Closed
Golodhros opened this issue Aug 11, 2020 · 3 comments
Closed

Add Storybook to the Frontend library #611

Golodhros opened this issue Aug 11, 2020 · 3 comments
Assignees
Labels
status:in_progress Issue that is being worked on right now type:feature A new feature request

Comments

@Golodhros
Copy link
Member

All projects create frontend components for their user interfaces. Amundsen has been creating some of these, and reusing others. However, it is not easy to understand what we have created before, the different styles we apply to them and if we have duplicates.

A storybook implementation will help us gather together our presentational components, making it easy to share, develop and evolve our UI, as well as keeping it consistent.

Expected Behavior or Use Case

We will have a storybook page with examples of our components.

Service or Ingestion ETL

Frontend Library

Possible Implementation

Storybook for React

@Golodhros Golodhros added type:feature A new feature request Project: Frontend status:in_progress Issue that is being worked on right now labels Aug 11, 2020
@dorianj
Copy link
Contributor

dorianj commented Aug 11, 2020

I'm looking into this

@dorianj
Copy link
Contributor

dorianj commented Aug 12, 2020

This is done

@Golodhros
Copy link
Member Author

Thanks @dorianj !

dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
* feat: Create ColumnType component (amundsen-io#604)

* Create ColumnType component

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Update Modal UI

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup; Add a test file

* Prevent ColumnListItem expand/collapse from being triggered

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* feat: Parse column types (amundsen-io#611)

* WIP: Create a parser & render parsed text

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Cleanup logic

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* More cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use more appropriate elements; Fix typo

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix button; Fix test; Remove obsolete style

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix duplicate test name

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* style: Improve UI styles and interactions (amundsen-io#617)

* Vertically center modal

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Match design font specifications

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Miscellaneous cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use variables

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* test: Improves unit tests for ColumnType + QA fixes (amundsen-io#625)

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Updates from design qa

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Improve ColumnType tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* log support

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix some lint warning

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Betterer update

Signed-off-by: Tamika Tannis <ttannis@lyft.com>
feng-tao pushed a commit that referenced this issue May 7, 2021
* feat: Create ColumnType component (#604)

* Create ColumnType component

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Update Modal UI

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup; Add a test file

* Prevent ColumnListItem expand/collapse from being triggered

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* feat: Parse column types (#611)

* WIP: Create a parser & render parsed text

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Cleanup logic

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* More cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use more appropriate elements; Fix typo

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix button; Fix test; Remove obsolete style

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix duplicate test name

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* style: Improve UI styles and interactions (#617)

* Vertically center modal

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Match design font specifications

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Miscellaneous cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use variables

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* test: Improves unit tests for ColumnType + QA fixes (#625)

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Updates from design qa

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Improve ColumnType tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* log support

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix some lint warning

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Betterer update

Signed-off-by: Tamika Tannis <ttannis@lyft.com>
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
* feat: Create ColumnType component (amundsen-io#604)

* Create ColumnType component

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Update Modal UI

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup; Add a test file

* Prevent ColumnListItem expand/collapse from being triggered

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* feat: Parse column types (amundsen-io#611)

* WIP: Create a parser & render parsed text

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Cleanup logic

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* More cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Lint fix

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use more appropriate elements; Fix typo

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix button; Fix test; Remove obsolete style

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix duplicate test name

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* style: Improve UI styles and interactions (amundsen-io#617)

* Vertically center modal

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Match design font specifications

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Miscellaneous cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Use variables

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* test: Improves unit tests for ColumnType + QA fixes (amundsen-io#625)

* Parser tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Updates from design qa

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Improve ColumnType tests

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* log support

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Code cleanup

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Fix some lint warning

Signed-off-by: Tamika Tannis <ttannis@lyft.com>

* Betterer update

Signed-off-by: Tamika Tannis <ttannis@lyft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:in_progress Issue that is being worked on right now type:feature A new feature request
Projects
None yet
Development

No branches or pull requests

2 participants