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

enhance(frontend): Better Timeline(MkPagination) Experience 2 #11241

Closed
wants to merge 126 commits into from

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Jul 11, 2023

What

  • 遡り中(backed)やタブがバックグラウンドの場合、追加されたノートのキューは全て溜め込んでおくように
  • スクロールを先頭に戻すと溜め込んだキューを順番に送り込んでいくように
  • その他いろいろめっちゃ変えた
  • Known Issueで書いた問題に対処するため、Safari(WebKit)とその他では挙動がちょっと変わっている

Additional

Safariだけ、DOMが先頭に追加されてもスクロールが調整されない
手動でスクロールを調整しても表示が崩れる

Safariでは遡りはせずに従来通りに最新のノートを表示することにした

@tamaina tamaina marked this pull request as draft July 11, 2023 07:58
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jul 11, 2023
@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Patch coverage: 97.27% and project coverage change: +0.09% 🎉

Comparison is base (814e284) 78.88% compared to head (b312f36) 78.98%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11241      +/-   ##
===========================================
+ Coverage    78.88%   78.98%   +0.09%     
===========================================
  Files          925      175     -750     
  Lines        97589    22957   -74632     
  Branches      7746      511    -7235     
===========================================
- Hits         76986    18132   -58854     
+ Misses       20603     4825   -15778     
Files Changed Coverage Δ
packages/frontend/src/scripts/scroll.ts 64.23% <62.50%> (+0.80%) ⬆️
...es/frontend/src/components/MkDateSeparatedList.vue 57.48% <88.88%> (-0.07%) ⬇️
packages/frontend/src/components/MkNotes.vue 100.00% <100.00%> (ø)
packages/frontend/src/components/MkPageWindow.vue 100.00% <100.00%> (ø)
packages/frontend/src/components/MkPagination.vue 99.06% <100.00%> (+0.25%) ⬆️
...ages/frontend/src/components/global/RouterView.vue 100.00% <100.00%> (ø)
packages/frontend/src/nirax.ts 64.78% <100.00%> (+5.57%) ⬆️
packages/frontend/src/scripts/useragent.ts 100.00% <100.00%> (ø)

... and 751 files with indirect coverage changes

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

@tamaina
Copy link
Contributor Author

tamaina commented Jul 13, 2023

なんかIntersectionObserverが効かなくなることがあって泥沼にはまってる感じ

@tamaina
Copy link
Contributor Author

tamaina commented Jul 14, 2023

IntersectionObserverは多分治った

@tamaina
Copy link
Contributor Author

tamaina commented Jul 14, 2023

trim.94633E27-DD3A-4710-9D8D-C3DE5CE649A8.MOV

Safariの描画の挙動が微妙
(HTMLElement.scrollでスクロールしても描画を省いているのが描画されない)

@tamaina
Copy link
Contributor Author

tamaina commented Jul 26, 2023

Safariで遡るのは諦めた

@tamaina
Copy link
Contributor Author

tamaina commented Jul 26, 2023

むしゃくしゃしたので昔の挙動と新しい挙動を設定で変えられるようにした

@tamaina tamaina mentioned this pull request Jul 26, 2023
5 tasks
@tamaina
Copy link
Contributor Author

tamaina commented Jul 29, 2023

noteViewInterruptorsって全ての表示(?)に対して適用していいのかしら

@syuilo
Copy link
Member

syuilo commented Sep 13, 2023

useScrollPositionManager とか RouterView 周りを変更した意図って何かしら

@tamaina
Copy link
Contributor Author

tamaina commented Sep 13, 2023

RouterViewはactivate後のタイムラインのスクロール位置の判定と旧useScrollPositionManagerの順番が噛み合わなかったために変えた

(たぶん他の場所でもこれに起因したバグが治ることがありそう)

@syuilo
Copy link
Member

syuilo commented Sep 13, 2023

「最新のノートに移動」したときに改めてAPIリクエストが発生するのは微妙かも

@tamaina
Copy link
Contributor Author

tamaina commented Sep 13, 2023

キューから持ってくるようにしても良さそう(キューから持ってくる実装を書くのが面倒だったためinitを呼び出すようにした)

@tamaina
Copy link
Contributor Author

tamaina commented Oct 5, 2023

改めてAPIリクエストが発生するのは微妙

リフレッシュも兼ねているので結構この実装のほうがいいかも

TLがプッシュ型になって早くなったらしいのでこのままでもいいのでは

@tamaina tamaina closed this Oct 22, 2023
@tamaina
Copy link
Contributor Author

tamaina commented Oct 22, 2023

コンフリクト解消を諦めたためclose

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants