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

Refactoring : Image > svg로 교체 #57

Merged
merged 10 commits into from
Sep 27, 2023
Merged

Conversation

guesung
Copy link
Collaborator

@guesung guesung commented Sep 12, 2023

💡 왜 PR을 올렸나요?

💁 무엇이 어떻게 바뀌나요?

  1. sprite를 이용하여 기존 Image태그에서 svg파일로 교체하였습니다.
    기존 Next/Image태그는 서버에 저장해둔 assets을 네트워크에서 불러와야했습니다. 로컬에서 가지고 있지 않고, 네트워크를 통해서 불러왔기에 속도가 다소 느렸습니다. 이에 따라 svg파일로 교체를 하고, sprite 라이브러리를 설치하였습니다.
    앞으로 svg파일을 추가하려면
  2. public>sprite>icons에 svg파일을 추가
  3. 터미널에 $yarn make-sprite를 치면
    public>sprite>sprite.svg에 자동으로 추가가 됩니다.

사용법은 다음과 같습니다.

<Icon id='icon-name' />

📆 작업 예정인 것이 있나요 ?

💬 리뷰어분들께

Copy link
Member

@seondal seondal left a comment

Choose a reason for hiding this comment

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

에셋 전부 바꾸는거라 꽤 복잡한 작업이였는데 대규모 리팩토링 수고하셨어요 ㅎㅎ
리뷰 남긴 이슈사항만 확인해주심 감사하겠습니다~!

src/app/(Sub)/menu/components/MakerSection.tsx Outdated Show resolved Hide resolved
src/components/Header/CloseButton.tsx Outdated Show resolved Hide resolved
src/components/Selection/Tag.tsx Outdated Show resolved Hide resolved
@guesung
Copy link
Collaborator Author

guesung commented Sep 27, 2023

리뷰 사항 수정완료했습니다 !

11bf8c7

close아이콘만 이상하게 sprite를 적용하면 fill이 흰색이 적용이 안되네요 ㅜㅜ 이 부분 제외하고 전부 Image > svg로 교체했습니다

Copy link
Member

@seondal seondal left a comment

Choose a reason for hiding this comment

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

확실히 속도가 빨라진게 느껴지네요 😆 짱! 수고하셨습니다

@guesung guesung merged commit 63931c7 into develop Sep 27, 2023
@guesung guesung deleted the feature/53-image-convert-icon branch September 27, 2023 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Image > svg 교체
2 participants