misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix(frontend): 瞬間的にstreamから切断された場合にもstream indicatorが出てしまうのを修正

Open kakkokari-gtyih opened this issue 1 year ago • 18 comments

What

一瞬切れただけでも出てしまうのはうざい可能性がある あと再接続されても消えなかったのでそれも修正

Why

Additional info (optional)

Checklist

  • [x] Read the contribution guide
  • [ ] Test working in a local environment
  • [ ] (If needed) Add story of storybook
  • [x] (If needed) Update CHANGELOG.md
  • [ ] (If possible) Add tests

kakkokari-gtyih avatar Oct 24 '24 14:10 kakkokari-gtyih

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 13.97%. Comparing base (cb03f3f) to head (8697628). :warning: Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14832       +/-   ##
============================================
- Coverage    62.86%   13.97%   -48.90%     
============================================
  Files         1150      237      -913     
  Lines       115224    11273   -103951     
  Branches      7907     3728     -4179     
============================================
- Hits         72439     1575    -70864     
+ Misses       40644     7578    -33066     
+ Partials      2141     2120       -21     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Oct 24 '24 14:10 codecov[bot]

コンフリクト解消

kakkokari-gtyih avatar Oct 25 '24 10:10 kakkokari-gtyih

コンフリクト解消

kakkokari-gtyih avatar Nov 15 '24 08:11 kakkokari-gtyih

  • 通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思うんだけどどうなんだろう
  • 一時的に切れた場合もその間にイベントを取りこぼしている可能性は少なくないから普通に通知した方が良い気がしないでもない

syuilo avatar Nov 15 '24 08:11 syuilo

通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思う

モバイル端末(モバイル回線が不安定な時)とか

一時的に切れた場合もその間にイベントを取りこぼしている可能性は少なくないから普通に通知した方が良い気がしないでもない

時限でおしらせするやつは無くしてconnected受け取ったら警告を自動で消すだけにするか

kakkokari-gtyih avatar Nov 15 '24 08:11 kakkokari-gtyih

connected受け取ったら警告を自動で消す

これはどういう意味があるかしら

syuilo avatar Nov 15 '24 08:11 syuilo

切断ダイアログが「現在切断されている通信がある」じゃなくて「過去に切断されたことがある」なので、新しいノートだけを見ている場合には単純に邪魔なので歓迎 細かい話では、再接続ダイアログとリアクションピッカーの当たり判定が突き抜けてて、リロードのつもりで誤リアクションすることがあって地味に困る

fruitriin avatar Nov 15 '24 08:11 fruitriin

細かい話では、再接続ダイアログとリアクションピッカーの当たり判定が突き抜けてて、リロードのつもりで誤リアクションすることがあって地味に困る

→ #14523

Sayamame-beans avatar Nov 15 '24 08:11 Sayamame-beans

connected受け取ったら警告を自動で消す

これはどういう意味があるかしら

切断ダイアログが「現在切断されている通信がある」じゃなくて「過去に切断されたことがある」なので、新しいノートだけを見ている場合には単純に邪魔なので歓迎

これ(仮にStreamの取りこぼしがあったとしても、通知/TLについてはPull To Refreshで補完できるようになっているのでライトなユースケースでは問題なさそう)

kakkokari-gtyih avatar Nov 15 '24 08:11 kakkokari-gtyih

よく理解できてないわね イベントを取りこぼした可能性があるなら通知した方が良いのではと思う

syuilo avatar Nov 15 '24 09:11 syuilo

よほど深刻な切断(長時間切断で数十個以上も抜けてるとか)じゃなかったら、切断っていうけど普通に動いてるじゃんとなるので…(消して欲しい) 一応通知系カラム(通知とダイレクト?)だけスクロール位置が一番上なら?自動リロード掛ければ致命的な見逃しを防げそうですが、そうすると欲が出てきそう (アンテナも自動でリロードして欲しいとかなってきてしまう)

Sayamame-beans avatar Nov 15 '24 09:11 Sayamame-beans

通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思うんだけどどうなんだろう

スマートフォン端末でよく瞬断を経験する気がする PCではあまり起きない感覚

Wifi→モバイルデータやその逆のタイミングなんかでもよく起きるかもしれない

tai-cha avatar Nov 15 '24 09:11 tai-cha

ずーーっと開きっぱなしにしていると時々PC(有線のデスクトップ)でも表示されているのを見かけますが、デスクトップだとそれぐらいかもですね(スリープとかすればそれは流石に切断されると考えられる)

Wifi→モバイルデータやその逆のタイミングなんかでもよく起きるかもしれない

同意します

Sayamame-beans avatar Nov 15 '24 09:11 Sayamame-beans

地下鉄乗ると出やすくていやん

fruitriin avatar Nov 15 '24 10:11 fruitriin

コンフリクト解消

kakkokari-gtyih avatar Feb 24 '25 05:02 kakkokari-gtyih

コンフリクト解消(次の次用)

kakkokari-gtyih avatar Dec 02 '25 02:12 kakkokari-gtyih

コンフリクト解消

kakkokari-gtyih avatar Dec 12 '25 04:12 kakkokari-gtyih

コンフリクト解消

kakkokari-gtyih avatar Dec 13 '25 09:12 kakkokari-gtyih