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

[Refactor/ProjectDetail/Performance] 성능 개선 #201

Merged
merged 38 commits into from
Apr 5, 2024

Conversation

Whoknow77
Copy link
Contributor

@Whoknow77 Whoknow77 commented Apr 4, 2024

#️⃣연관된 이슈

#200

💡 핵심적으로 구현된 사항

  • 이미지 지연 로딩
  • 코드 분할
  • 텍스트 압축
  • 라우터 구조 수정
  • react-icons 필요한것만 받아오도록 수정
  • img태그 alt속성 추가
  • 대비 색상 수정

➕ 그 외에 추가적으로 구현된 사항

🤔 테스트,검증 && 고민 사항

  • 코드 분할을 적용하다가 RootLayout 파일이 필요없다고 생각해서 빼고 라우터 구조를 중첩 라우팅으로 수정했습니다. prefetch도 잘 되는거같은데 확인 한번 부탁드립니다.
  • lighthouse 기준 성능약간, 접근성, 권장사항 정도를 건드려봤는데 배포환경에서 확인해야해서 끊어서 확인하려고 합니다. 프로젝트 상세페이지 기준으로 개선한거라서 각자 페이지에서 확인이 필요합니다.(제가 건드린게 전체적으로 건드린 부분도 있겠지만 아닌 부분도 꽤 있을 겁니다)

📌 PR Comment 작성 시 Prefix for Reviewers

@Whoknow77 Whoknow77 added the Refactor 리팩토링 label Apr 4, 2024
@Whoknow77 Whoknow77 self-assigned this Apr 4, 2024
Copy link
Contributor

@wdgWon wdgWon left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!
200줄 넘게 리팩토링하시는게 정말 대단하신것 같아요 ㄷㄷ
저도 반응형하고 리드미 해야되는데 손이 안떨어지네요 ㅜㅜ

index.html Outdated Show resolved Hide resolved
src/routes/index.tsx Show resolved Hide resolved
src/routes/layouts/DefaultLayout.tsx Outdated Show resolved Hide resolved
@Whoknow77 Whoknow77 force-pushed the refactor/#200/ProjectDetail/performance branch from 93014f5 to 8b50a10 Compare April 5, 2024 07:03
@Whoknow77 Whoknow77 merged commit 967552c into dev Apr 5, 2024
Whoknow77 added a commit that referenced this pull request Apr 5, 2024
…rformance2

[Refactor/#201/project detail/performance2] 검색엔진 오류 수정
@Whoknow77 Whoknow77 deleted the refactor/#200/ProjectDetail/performance branch April 5, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants