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

docs: BoxButton, TextButton 컴포넌트 문서에 아이콘 관련 내용을 추가 #155

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

nijuy
Copy link
Collaborator

@nijuy nijuy commented Sep 5, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

기존 코드에 영향을 미치지 않는 변경사항

  • iconBasicDisabledStrong 토큰을 추가했습니다

  • Boxbutton TextButton 내부 svg의 컬러를 fill로 제어하도록 수정했습니다

    기존에는 텍스트 컬러 지정을 위해 넣은 color 속성이 svg에도 적용되고 있었는데요

    이는 text semantic color를 아이콘에도 사용하는 거랑 같아서.... fill로 svg 색상을 명시했습니다

  • Boxbutton TextButton 문서에 leftIcon(rightIcon) 관련 내용을 추가했습니다

    image

    image

2️⃣ 알아두시면 좋아요!

3️⃣ 추후 작업

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

- leftIcon, rightIcon을 삽입하는 경우를 문서 에 추가
- leftIcon, rightIcon을 삽입하는 경우를 문서에 추가
@nijuy nijuy added the docs label Sep 5, 2024
@nijuy nijuy self-assigned this Sep 5, 2024
Copy link
Member

@seocylucky seocylucky left a comment

Choose a reason for hiding this comment

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

확인했습니당!!👍🏻👍🏻 알잘딱깔센으로 이미 코드를 적어둔 보리 그저 갓

@nijuy
Copy link
Collaborator Author

nijuy commented Sep 19, 2024

@seocylucky ㅋㅋㅋㅋ 그냥 문서 덜 본 사람인데 이.. 이런 칭찬.
근데 제가 못 본 피그마 코멘트가 있어서 다시 수정하고 리뷰 넣을게요!!
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: BoxButton TextButton 내부 아이콘 색상 지정 방식 수정
2 participants