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

fix: 책 상세 진입때 스크롤 관련 버그 #642

Merged
merged 4 commits into from
Nov 24, 2024

Conversation

not-using
Copy link
Collaborator

작업내용

책 상세 페이지 진입시 스크롤 문제

  • 문제
    • 이전 페이지에서 스크롤한 상태일 때, 특히 메인에서 진입시 스크롤 이상함
    • 데스크탑 : 스크롤 중간쯤
      모바일 : 스크롤 최상단
    • breadCrumb 쪽에 scrollIntoView 코드가 들어있지만 제대로 동작하지 않음
  • 원인 및 해결
    • 책정보 api가 호출 되기 전 랜더링된 화면에는 scrollIntoView에 연결된 ref가 없음
    • ref.current를 의존성에 추가, ref가 이후 마운트 되더라도 영향을 받도록

추가 스크롤 오류

  • 뒤로가기 클릭시 스크롤이 초기화되는 문제
    • 도서 목록등에서 보고있는 스크롤 위치로 돌아가지 않음
      -> 책 상세 진입시 이전 화면의 스크롤 위치를 기억해두고 돌아갈때 다시 설정되도록
  • 도서목록으로 진입시 기존스크롤에 영향받는 문제
    • 모바일 화면에서만 문제가 됨
    • 데스크탑은 헤더가 최상단에서만 보임, 기존 스크롤 위치가 문제되지 않음
      -> 모바일 헤더버튼에 최상단으로 스크롤 올리는 로직 추가
before after
before Nov-17-2024 21-00-51

@not-using not-using self-assigned this Nov 17, 2024
Copy link
Contributor

@mixsung mixsung left a comment

Choose a reason for hiding this comment

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

로딩 중 빈 공간 마련을 위해 isFetched가 존재하는군요 좋습니다!

@not-using not-using force-pushed the 518-스크롤-버그 branch from a9220dd to dce5464 Compare November 24, 2024 07:17
@not-using not-using merged commit 3a1cddf into develop Nov 24, 2024
4 checks passed
@not-using not-using deleted the 518-스크롤-버그 branch November 24, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants