-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Refactor] 배럴 파일 삭제 작업 #324
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.
고생하셨습니다! 역시 최적화의 달인 ㅎ
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.
SVG 파일 많아서 변경하기 힘들었을텐데 수고하셨습니다!
lazy로 코드 스플리팅을 하기 위해 배럴 파일을 삭제하게 되었는데 import문이 지저분해진다는 단점 외에도 혹시 이렇게 변경했을 때 단점이 또 있을까요?
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.
감사합니다 ! 덕분에 Barrel 파일도 성능최적화도, 웹에 대해 새롭게 많이 알아가는 것 같습니다:) 👍
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.
고생하셨습니다~~ 대쉬 최적화 시작~~!!
없습니다!! |
📌 Related Issues
✅ 체크 리스트
📄 Tasks
배럴 파일 삭제(이미지, svg)
현재 svg컴포넌트를 배럴 파일(index.ts)을 사용해서 여러 컴포넌트를 한 곳에서 export함으로써 편리성을 제공한다.(+깔끔한 import문), 그러나 빌드 시점에 배럴 파일이 이미 번들에 포함되기 때문에 svg컴포넌트에 lazy를 적용해도 적용되지 않는 문제가 발생한다.
assets하위에 존재하던 이미지 파일 모두 public으로 옮기기
모든 페이지에 적용되어있던 lazy 삭제
⭐ PR Point (To Reviewer)
📷 Screenshot
🔔 ETC