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

[Feat] panda preset 을 배포할 수 있는 있도록 wow-theme 패키지 추가 #111

Merged
merged 7 commits into from
Aug 4, 2024

Conversation

SeieunYoo
Copy link
Collaborator

@SeieunYoo SeieunYoo commented Aug 1, 2024

🎉 변경 사항

  • preset 에서 다룰 수 있는 옵션들을 아래와 같아서 staticCSS, theme 만 다루는 것으로 했습니다.
    image

  • panda preset 을 배포하는 패키지만 만들까 하다가 theme/utils 도 배포하면 좋을 거 같아서 theme 패키지를 배포하는 것으로 구현했습니다

  • rollup 설정은 wowds-token 을 참고했습니당

  • 진입점 index.js vs 진입점 util,preset

    • panda config 정의할 때 theme/utils 와 theme/preset 모두 다 쓸 거 같아서 index.js 로 통일
    • 진입점 index.js
      image
    • 진입점 utils,preset
      image

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

  • preset 만 배포하는 패키지로 분리해야될 것 같다면 수정하겠습니당
  • panda config 정의할 때 theme/utils 와 theme/preset 모두 다 쓸 거 같아서 index.js 로 통일했는데 진입점 분리 필요하면 말씀 주세용

Copy link
Collaborator

@ghdtjgus76 ghdtjgus76 left a comment

Choose a reason for hiding this comment

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

굿굿 수고하셨어요~!

진입점은 굳이 분리하지 않아도 될 거 같다는 생각이 들어요!
제가 컴포넌트별로 진입점을 분리한 이유는 컴포넌트를 불러올 때 항상 모든 컴포넌트를 사용하지 않을 때도 다 불러와서 분리했던 거고, 여기서는 둘 다 사용할 거 같아서 분리 안 하고 사용해도 될 거 같습니다~

packages/wow-theme/src/preset/preset.ts Outdated Show resolved Hide resolved
packages/wow-theme/src/tokens/index.ts Outdated Show resolved Hide resolved
packages/wow-theme/package.json Outdated Show resolved Hide resolved
@SeieunYoo
Copy link
Collaborator Author

@ghdtjgus76 리뷰 반영해두었습니다!!!

@SeieunYoo SeieunYoo requested a review from ghdtjgus76 August 3, 2024 07:11
Copy link
Collaborator

@eugene028 eugene028 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 서현님 의견처럼 wow-theme 한해서 utils만을 따로 분리하여 사용하지 않을 것 같기도 하고, 파일의 크기차이도 영향력이 적을 것 같아서 진입점을 index.js로 같이 해도 될 것 같아요

packages/wow-theme/tsconfig.json Outdated Show resolved Hide resolved
packages/wow-theme/src/types/Color.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@ghdtjgus76 ghdtjgus76 left a comment

Choose a reason for hiding this comment

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

굿굿 수고 많으셨어요~

@SeieunYoo SeieunYoo merged commit 611e633 into main Aug 4, 2024
@SeieunYoo SeieunYoo deleted the feat/panda-preset branch August 4, 2024 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] panda preset 배포하는 패키지 추가
3 participants