Skip to content

Conversation

@yas-ako
Copy link
Contributor

@yas-ako yas-ako commented Dec 5, 2025

概要

メッセージ読み込み直後, MessagesScroller.vue の watch によるスクロール位置調整が行われているとき、useMessageScrollerElementResizeObserver.ts による位置調整を無効化することで、別々の場所でレイアウト調整が同時に行われることを防ぐ

なぜこの PR を入れたいのか

close #4976

動作確認の手順

UI 変更部分のスクリーンショット

before

after

PR を出す前の確認事項

  • (機能の追加なら)追加することの合意がチームで取れている
    • 取れていない場合はチェックを外して PR にすれば OK
  • 動作確認ができている
  • 自分で一度コードを眺めて自分的に問題はなさそう

見てほしいところ・聞きたいことなど

画像がたくさん投稿されたチャンネルで問題がなさそうなことを確認した
問題が起きていない下方向へのスクロールについては、正しい挙動を維持できていそう
他のバグを引き起こしていないか確かめる必要がある

@yas-ako yas-ako self-assigned this Dec 5, 2025
@github-actions
Copy link

github-actions bot commented Dec 5, 2025

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.89%. Comparing base (88bf296) to head (c972a78).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4977   +/-   ##
=======================================
  Coverage   60.89%   60.89%           
=======================================
  Files         102      102           
  Lines        2969     2969           
  Branches      614      614           
=======================================
  Hits         1808     1808           
  Misses       1067     1067           
  Partials       94       94           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yas-ako yas-ako requested a review from cp-20 December 7, 2025 06:35
Copy link
Contributor

@cp-20 cp-20 left a comment

Choose a reason for hiding this comment

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

ありがとう

軽く動作確認した感じは大丈夫そうなので Approve しておきます

欲を言えば e2e テストとかをかけると良いですね

@yas-ako
Copy link
Contributor Author

yas-ako commented Dec 7, 2025

依然としてスクロール位置の調整を行うコードが別々の場所に分かれたままになってしまっているので、一箇所にまとめたい

@yas-ako yas-ako merged commit 561f1c6 into master Dec 7, 2025
11 checks passed
@yas-ako yas-ako deleted the fix/keep-message-scroll-position branch December 7, 2025 14:15
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.

より古いメッセージを新たに読み込んだとき、スクロール位置が下にずれてしまう

3 participants