bug: color.stories.ts semanticColorCategories 수정 및 AllThemeColors 에러 해결 작업 #176
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fecapark
force-pushed
the
fix/#175-color-docs-fix
branch
from
5251371 to
d2fac76
Compare
seocylucky
approved these changes
Jan 25, 2025
Member
seocylucky
left a comment
seocylucky
left a comment
There was a problem hiding this comment.
해당 오류난 부분에서 이제 문제는 없는거 같으니 어푸룹하고 머지할게여!
seocylucky
changed the title
seocylucky
added a commit
that referenced
this pull request
Jan 25, 2025
* feat: LoadingIndicator.style 생성 * feat: LoadingIndicatorProps 타입 생성 * feat: LoadingIndicator 컴포넌트 구현 * docs: LoadingIndicator 문서 작성 * chore: components/index.ts에 LoadingIndicator export문 추가 * Fix/#175 color docs fix (#176) * fix: semanticColorCategories 수정 * fix: semanticColorCategories 수정 * 🐛 AllThemeColors 컴포넌트가 스토리에 문서 전용 컴포넌트 렌더링하던 문제 수정 * fix: semanticColorCategories snackbar 오타 수정 --------- Co-authored-by: seocylucky <[email protected]> * chore: update version v2.0.1 (#177) --------- Co-authored-by: nijuy <[email protected]> Co-authored-by: Sanghyeok Park <[email protected]>
fecapark
added a commit
that referenced
this pull request
Jan 28, 2025
* fix: semanticColorCategories 수정 * fix: semanticColorCategories 수정 * 🐛 AllThemeColors 컴포넌트가 스토리에 문서 전용 컴포넌트 렌더링하던 문제 수정 * fix: semanticColorCategories snackbar 오타 수정 --------- Co-authored-by: seocylucky <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1️⃣ 어떤 작업을 했나요? (Summary)
기존 코드에 영향을 미치지 않는 변경사항
기존 코드에 영향을 미치는 변경사항
semanticColorCategories에 있는 애들이 semanticColorPalette에 존재하지 않아 getSemanticColorPalette에서 색상을 찾을 수 없어 나타난 오류였습니다. semanticColorPalette와 semanticColorCategories를 맞춰주었어요!
AllThemeColors컴포넌트에서 문서 전용 컴포넌트인ColorPalette,ColorItem을 렌더링하고 있었어요.따라서
AllThemeColors의 스토리에서 해당 컴포넌트들을 렌더링할 수 없는 문제가 있었던 거고요.그래서 기존
AllThemeColors에서 가져오던 시맨틱 컬러셋들을 render props 패턴으로 사용처로 렌더링 작업을 위임하고,문서를 렌더링하는 사용처에서
ColorPalette,ColorItem을 사용해서 렌더링 작업을 해주도록 변경했어요.살짝 아쉬운거는
AllThemeColors스토리가 이렇게 보인다는 점...? 근데 애초에 이게 정상 작동이니 큰 문제는 없을듯해요버그 픽스
2️⃣ 알아두시면 좋아요!
3️⃣ 추후 작업
4️⃣ 체크리스트 (Checklist)
main브랜치의 최신 코드를pull받았나요?