-
Notifications
You must be signed in to change notification settings - Fork 3
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
에셋 전부 바꾸는거라 꽤 복잡한 작업이였는데 대규모 리팩토링 수고하셨어요 ㅎㅎ
리뷰 남긴 이슈사항만 확인해주심 감사하겠습니다~!
리뷰 사항 수정완료했습니다 ! close아이콘만 이상하게 sprite를 적용하면 fill이 흰색이 적용이 안되네요 ㅜㅜ 이 부분 제외하고 전부 Image > svg로 교체했습니다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확실히 속도가 빨라진게 느껴지네요 😆 짱! 수고하셨습니다
💡 왜 PR을 올렸나요?
💁 무엇이 어떻게 바뀌나요?
기존 Next/Image태그는 서버에 저장해둔 assets을 네트워크에서 불러와야했습니다. 로컬에서 가지고 있지 않고, 네트워크를 통해서 불러왔기에 속도가 다소 느렸습니다. 이에 따라 svg파일로 교체를 하고, sprite 라이브러리를 설치하였습니다.
앞으로 svg파일을 추가하려면
$yarn make-sprite
를 치면public>sprite>sprite.svg에 자동으로 추가가 됩니다.
사용법은 다음과 같습니다.
📆 작업 예정인 것이 있나요 ?
💬 리뷰어분들께