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

[Chat] iOS 모바일에서 init 시점에 infinite scroll 로직 실행되는 오류 수정 #2868

Merged
merged 8 commits into from
Aug 30, 2023

Conversation

dia-triple
Copy link
Contributor

@dia-triple dia-triple commented Aug 28, 2023

PR 설명

  • 채팅 infinite scroll 오류를 수정합니다.
    • 채팅방이 Popup과 함께 쓰이는 경우, iOS 모바일 사파리에서 init 시점에 infinite scroll이 트리거 되어 bottom으로 이동하는 스크롤과, top으로 이동하는 스크롤이 중첩되는 현상이 있습니다.
      • IntersectionObserver의 time은 도큐먼트가 생성된 시점으로 부터의 시간이므로, 대부분의 경우 MINIMUM_INTERSECTING_TIME보다 항상 큽니다.
      • iOS 모바일 사파리에서 Intersecting이 팝업 오픈 transform이 모두 완료된 이후 true로 변경되어(iOS 모바일 사파리에서는 Intersecting 상태가 transform에 영향을 받는 것으로 예상됨, 관련 이슈) init 시점에 bottom으로 이동하는 스크롤과, infinite scroll이 트리거 되면서 top으로 이동하는 스크롤이 중첩되어 스크롤이 부자연스러운 오류가 발생합니다.

변경 내역

  • infinite scroll 실행 조건을 Intersecting time 비교에서 isScrollReady 플래그로 변경했습니다. 최초 메시지 로드 후 bottom으로 이동하는 커멘드가 실행되면 isScrollReady가 true로 변경됩니다.

체크리스트

스크린샷 & URL

@codecov
Copy link

codecov bot commented Aug 28, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (2644748) 12.33% compared to head (91b06ad) 12.32%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2868      +/-   ##
==========================================
- Coverage   12.33%   12.32%   -0.01%     
==========================================
  Files         676      676              
  Lines       35766    35770       +4     
  Branches     8817     8820       +3     
==========================================
  Hits         4410     4410              
- Misses      31350    31354       +4     
  Partials        6        6              
Files Changed Coverage Δ
packages/chat/src/chat/chat.tsx 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dia-triple
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.2.5-pr-2868.3 has been published!

@dia-triple dia-triple force-pushed the feat/chat-inspection branch 3 times, most recently from de20bcd to 6871a8f Compare August 29, 2023 06:05
@dia-triple
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.2.5-pr-2868.21 has been published!

@dia-triple dia-triple force-pushed the feat/chat-inspection branch from 6871a8f to b41e078 Compare August 29, 2023 06:43
@dia-triple
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.2.5-pr-2868.23 has been published!

@dia-triple
Copy link
Contributor Author

release-canary

@github-actions
Copy link

v13.2.5-pr-2868.24 has been published!

@dia-triple dia-triple self-assigned this Aug 29, 2023
@dia-triple dia-triple added this to the v13.3.0 milestone Aug 30, 2023
@dia-triple dia-triple marked this pull request as ready for review August 30, 2023 04:40
@dia-triple dia-triple requested a review from a team as a code owner August 30, 2023 04:40
@dia-triple dia-triple requested review from inbeom, choisohyun and guswl98 and removed request for a team August 30, 2023 04:40
@github-actions github-actions bot added the chat label Aug 30, 2023
@dia-triple dia-triple merged commit b907626 into main Aug 30, 2023
@dia-triple dia-triple deleted the feat/chat-inspection branch August 30, 2023 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants