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

Gutenberg: contribute story for menu-group #78

Closed
bangank36 opened this issue Jul 27, 2023 · 3 comments
Closed

Gutenberg: contribute story for menu-group #78

bangank36 opened this issue Jul 27, 2023 · 3 comments

Comments

@bangank36
Copy link
Owner

bangank36 commented Jul 27, 2023

Summary

Use this issue to track the process of making story for Gutenberg menu-group item

There is story of menu-group''s child menu-item-choices

Currently all components stories are using addon-knobs, I think we should start using addon-controls for this component

🚧 Proposed PR:

MenuGroup: Add Storybook stories

  • 28-July-2023: waiting for review

Get started with Gutenberg

Reference

Initial discussion on the contribution
Storybook docs

@bangank36 bangank36 converted this from a draft issue Jul 27, 2023
@bangank36
Copy link
Owner Author

bangank36 commented Jul 27, 2023

27 July 2023

  1. label
  2. dividers
  • Current state of story
    • To add description of the props we need type file with detail comment
    • For generated code to copy we need to return the Template
    • For ColorPalette, the CSS variables showing Template component, does not make sense?
    • First item with example code can be found on main file of menu group
    • meta: ComponentMeta< typeof ButtonGroup > will expect type of component to parse the prop property

@bangank36
Copy link
Owner Author

bangank36 commented Jul 28, 2023

28 July 2023

  • Storybook docs only works if import from the default export of component that is relative to the stories, import directly from wordpress/components did not work
    import MenuGroup from '..'; > import { MenuGroup } from '@wordpress/components';

Image

Can not commit

✖ node ./bin/packages/lint-staged-typecheck.js found some errors. Please fix them and try committing again.

error TS5055: Cannot write file '/Users/nganle/Documents/Automattic/gutenberg/packages/components/build-types/alignment-matrix-control/icon.d.ts' because it would overwrite input file.

Following this https://togithub.com/WordPress/gutenberg/pull/18942 to fix, some thing wrong with write permission?
Use internal import and re-run npm run clean:package-types helps

@github-project-automation github-project-automation bot moved this from Triage to Done in Personal Development Jul 28, 2023
@bangank36 bangank36 reopened this Jul 28, 2023
@bangank36 bangank36 moved this from Done to In Progress in Personal Development Jul 28, 2023
@bangank36
Copy link
Owner Author

Closed as PR merged

@github-project-automation github-project-automation bot moved this from In Progress to Done in Personal Development Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

1 participant