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

[#25] 사용자가이드 단계 이동시 헤더 내 모든 요소 리렌더링 개선 #37

Merged
merged 8 commits into from
Jan 24, 2025

Conversation

chichoc
Copy link
Collaborator

@chichoc chichoc commented Jan 24, 2025

🔗 #25

🙋‍ Summary (요약)

  • 컴포넌트 분리 및 메모이제이션
  • 4단계만 메모이제이션 커스터마이징

😎 Description (변경사항)

사용자 가이드 단계 이동시 해당 단계와 관계없는 요소들이 리렌더링되고 있다.
리렌더링_사용자가이드

헤더부분 리렌더링 개선된 결과는 다음과 같다.
사용자가이드헤더개선
헤더쪽 테두리는 헤더가 아닌 workspacePage가 리렌더링되어 보여지는 것이다.

컴포넌트 분리 및 메모이제이션

  • 도움말 버튼 분리
  • 내부 요소 memo 활용하여 메모이제이션

워크스페이스 헤더 리렌더링 개선

  • 단계 이동시 4단계거나 4단계였던 경우만 리렌더링 되도록 memo 커스터마이징

🔥 Trouble Shooting (해결된 문제 및 해결 과정)

🤔 Open Problem (미해결된 문제 혹은 고민사항)

CssPropsSelectBox와 PreviewBox 리렌더링은 아직 개선하지 못했습니다.
#35 머지되면 다시 체크해보겠습니다.

@chichoc chichoc added the refactor 리팩토링 label Jan 24, 2025
@chichoc chichoc self-assigned this Jan 24, 2025
@chichoc chichoc requested a review from a team as a code owner January 24, 2025 06:21
@chichoc chichoc requested review from lee0jae330 and inhachoi and removed request for a team January 24, 2025 06:21
Copy link
Member

@lee0jae330 lee0jae330 left a comment

Choose a reason for hiding this comment

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

useShallow을 통해 필요한 상태만 구독하는 방법을 공유해주셔서 좋았습니다 ㅎㅎ 만약, 리렌더링이 개선이 안된다면, 코치마크 레이어의 특정 부분만 투명하게 하는 방법을 도입해봐도 좋을 것 같네요. (실현가능성은 잘 모르겠습니다..ㅎ)
고생하셨습니다. <3

Copy link
Contributor

@inhachoi inhachoi left a comment

Choose a reason for hiding this comment

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

리렌더링 개선을 하셨군요!
memo 적용하신거 좋네요!
수고 많으셨습니다 ❤️

@inhachoi inhachoi merged commit 76188d7 into dev Jan 24, 2025
5 checks passed
@chichoc chichoc linked an issue Jan 25, 2025 that may be closed by this pull request
4 tasks
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.

[refactor] 사용자가이드 리렌더링 개선
3 participants