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

[FE] FIX: 라이트하우스 성능 개선 #1687 #1704

Merged
merged 16 commits into from
Nov 13, 2024
Merged

Conversation

jnkeniaem
Copy link
Collaborator

@jnkeniaem jnkeniaem commented Nov 5, 2024

해당 사항 (중복 선택)

  • FEAT : 새로운 기능 추가 및 개선
  • FIX : 기존 기능 수정 및 정상 동작을 위한 간단한 추가, 수정사항
  • BUG : 버그 수정
  • REFACTOR : 결과의 변경 없이 코드의 구조를 재조정
  • TEST : 테스트 코드 추가
  • DOCS : 코드가 아닌 문서를 수정한 경우
  • REMOVE : 파일을 삭제하는 작업만 수행
  • RENAME : 파일 또는 폴더명을 수정하거나 위치(경로)를 변경
  • ETC : 이외에 다른 경우 - 어떠한 사항인지 작성해주세요.

설명

HOME

Eliminate render-blocking resources - Potential savings of 280 ms

  • first paint를 지연시키는 리소스들을 defer속성을 사용하기

Avoid large layout shifts - 1 layout shift found

  • 홈 페이지 접속시 로딩 애니메이션이 움직이지 않게 위치 고정

Serve images in next-gen formats

  • Coin image를 png형식이 아니라 svg로 가져오기

폰트 전체를 import 하는 대신 필요한 부분만 import

  • 384kb 에서 72.5kb 로 80% 이상 크기 최적화

#1687

설명 링크

https://www.notion.so/hyunja/Chrome-Lighthouse-7e66955ceaed42098b5ce8111273439d

@jnkeniaem jnkeniaem linked an issue Nov 5, 2024 that may be closed by this pull request
@jnkeniaem jnkeniaem changed the title Fe/dev/lighthouse#1687 [FE] FIX: 라이트하우스 성능 개선 #1687 Nov 5, 2024
@jnkeniaem jnkeniaem added FEAT New feature or upgrade FE Frontend tasks FIX Fix former works better labels Nov 5, 2024
@jnkeniaem jnkeniaem marked this pull request as ready for review November 5, 2024 07:23
Copy link
Collaborator

@seonmiki seonmiki left a comment

Choose a reason for hiding this comment

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

아직은 구현하기에만 급급해서 성능 최적화에 대해서 제대로 고민하고 개선해본 적이 없는데, 정리해주신 글 보면서 많이 배워갑니다! pr에 요약해주신 부분 말고도 lazy import 나 robots.txt 같은 부분들도 다 처음보는데 신기하고 재밌네요~ 담에 기회가 된다면 다른 부분들도 더 개선할 방법을 찾아보면 좋을 것 같습니다ㅎㅎ 고생 많으셨습니다!!

Copy link
Collaborator

@wet6123 wet6123 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ 정리한 내용 같이 넣어주셔서 이해하기가 한결 편했습니다. 라이트 하우스 관련해서도 새롭게 배울 내용이 많네요. 이번 기회에 저도 한번 사용해봐야겠습니다!

@jnkeniaem jnkeniaem merged commit 36e7133 into dev Nov 13, 2024
1 check passed
@jnkeniaem jnkeniaem deleted the fe/dev/lighthouse#1687 branch November 13, 2024 06:32
@Minkyu01 Minkyu01 restored the fe/dev/lighthouse#1687 branch November 15, 2024 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE Frontend tasks FEAT New feature or upgrade FIX Fix former works better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] LIGHTHOUSE 성능 개선
6 participants