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

Chore: Figma SVG 추출 #25

Merged
merged 3 commits into from
Jul 29, 2024
Merged

Chore: Figma SVG 추출 #25

merged 3 commits into from
Jul 29, 2024

Conversation

JisuuungKim
Copy link
Member

#️⃣연관된 이슈

#13

📝작업 내용

public/svgs 폴더에 파일을 저장했습니다

저장한 SVG 아이콘 목록 및 설명

  1. 현재 작업되어있는 아이콘 컴포넌트
image fill=on,off 가 존재하는 아이콘의 경우, fill=on인 아이콘의 이름에 fill을 붙여 저장했습니다. ex. heart-fill.svg
  1. 소셜 로그인 관련 아이콘
  1. 메인페이지의 위시리스트, 마이페이지 아이콘
image

아이콘의 이름에 color를 붙여 저장했습니다.
ex. heart-color.svg

추가 설명 사항

  • 현재 페이지에 작업되어 있는 아이콘들은 모두 아이콘 컴포넌트에 있는 것들로 대체될 예정이라고 하여 아이콘 컴포넌트에 있는 svg파일만 저장했습니다.
  • 아래 컴포넌트의 빨간색 테두리로 표시한 아이콘들은 뺄지 말지 고민중이라고 합니다.
image - svg파일을 컴포넌트로 활용하기 위해 public/svgs/index.ts에 export 작업을 해두었습니다.

@JisuuungKim JisuuungKim added the 👏 enhancement 새로운 기능 및 요청 label Jul 22, 2024
@JisuuungKim JisuuungKim self-assigned this Jul 22, 2024
@JisuuungKim JisuuungKim linked an issue Jul 22, 2024 that may be closed by this pull request
3 tasks
Copy link
Member

@j-myeong j-myeong left a comment

Choose a reason for hiding this comment

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

특별한 문제 없어보여서 Approve 진행하고, 추후에 이름이 겹치는 문제 등이 발생하면 추가적으로 이슈에 남겨서 진행하도록 하면 좋을 것 같습니다.

고생하셨습니다!

@JisuuungKim JisuuungKim merged commit 5aff7c0 into develop Jul 29, 2024
@j-myeong j-myeong deleted the chore/13-svg-setting branch July 29, 2024 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👏 enhancement 새로운 기능 및 요청
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: Figma SVG 추출
2 participants