feat: handy icon 적용 #121
Merged
feat: handy icon 적용 #121
Conversation
This file contains 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
- 새 아이콘 svg 추가 - 피그마 내 `icChat` 이름 중복으로 2개 누락된 상태
- 기존 아이콘을 사용 중인 컴포넌트가 있어 스토리북 에러 발생 - 어차피 비울 예정이기도 했음
- icon context가 삭제되었기 때문에 size, color 기본값을 IconBase에 설정
nijuy
commented
Jul 9, 2024
|
|
||
| ```typescript | ||
| import { IcSearchLine, IcShareLine, IconContext } from '@yourssu/design-system-react'; | ||
| import { IcAddFilled } from '@yourssu/design-system-react'; |
There was a problem hiding this comment.
여기 아직 이전 이름으로 남아있는데요. 여쭤보고 싶은 게 있어서 수정 안 해뒀습니다
원래는 @조직/이름 형식으로 지어서 YDS임을 나타내고 있었는데요
디자인시스템 이름을 Yourssu Design System -> Handy Design System 으로 바꾸기로 했잖아요!
그럼 배포시 이름을 handy-design-system-react으로 수정하는 걸 생각하신 게 맞나요?
13 tasks
fecapark
approved these changes
Jul 10, 2024
- 기존 icChat -> icComment - 누락됐던 icChat 추가
- package.json name 수정하면서 여기도 같이 고쳐야 함
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)
기존 코드에 영향을 미치는 변경사항
component하위 내용을 일부 삭제했습니다. (YDSWrapper만 남김)기존 아이콘을 사용하고 있는 컴포넌트들이 있어서 안 지우면 에러 나요 싹 다 비워버림
아이콘 문서 내용을 더 자세하게 수정했습니다
2️⃣ 알아두시면 좋아요!
아이콘 2개가 이름 중복 문제로 누락된 상태인데 답변 오면 추가해둘게요 (관련 스레드)
PR 올리고 나니까 svg를 react component처럼 사용하도록 하면 안되냐고 했던 게 생각났어요 미쳣나!!!!!!!!
이거 하려면
svgr써야 하는데 (저번 회의에서 그렇게 하려면 뭔가,,, 필요하다. 라고 했던 그거) 낼 찾아볼게욥 죄송.3️⃣ 추후 작업
4️⃣ 체크리스트 (Checklist)
main브랜치의 최신 코드를pull받았나요?