fix(frontend): 瞬間的にstreamから切断された場合にもstream indicatorが出てしまうのを修正
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
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.
コンフリクト解消
コンフリクト解消
- 通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思うんだけどどうなんだろう
- 一時的に切れた場合もその間にイベントを取りこぼしている可能性は少なくないから普通に通知した方が良い気がしないでもない
通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思う
モバイル端末(モバイル回線が不安定な時)とか
一時的に切れた場合もその間にイベントを取りこぼしている可能性は少なくないから普通に通知した方が良い気がしないでもない
時限でおしらせするやつは無くしてconnected受け取ったら警告を自動で消すだけにするか
connected受け取ったら警告を自動で消す
これはどういう意味があるかしら
切断ダイアログが「現在切断されている通信がある」じゃなくて「過去に切断されたことがある」なので、新しいノートだけを見ている場合には単純に邪魔なので歓迎 細かい話では、再接続ダイアログとリアクションピッカーの当たり判定が突き抜けてて、リロードのつもりで誤リアクションすることがあって地味に困る
細かい話では、再接続ダイアログとリアクションピッカーの当たり判定が突き抜けてて、リロードのつもりで誤リアクションすることがあって地味に困る
→ #14523
connected受け取ったら警告を自動で消す
これはどういう意味があるかしら
切断ダイアログが「現在切断されている通信がある」じゃなくて「過去に切断されたことがある」なので、新しいノートだけを見ている場合には単純に邪魔なので歓迎
これ(仮にStreamの取りこぼしがあったとしても、通知/TLについてはPull To Refreshで補完できるようになっているのでライトなユースケースでは問題なさそう)
よく理解できてないわね イベントを取りこぼした可能性があるなら通知した方が良いのではと思う
よほど深刻な切断(長時間切断で数十個以上も抜けてるとか)じゃなかったら、切断っていうけど普通に動いてるじゃんとなるので…(消して欲しい) 一応通知系カラム(通知とダイレクト?)だけスクロール位置が一番上なら?自動リロード掛ければ致命的な見逃しを防げそうですが、そうすると欲が出てきそう (アンテナも自動でリロードして欲しいとかなってきてしまう)
通常の環境であれば「瞬間的にstreamが切れる」というのは発生しないと思うんだけどどうなんだろう
スマートフォン端末でよく瞬断を経験する気がする PCではあまり起きない感覚
Wifi→モバイルデータやその逆のタイミングなんかでもよく起きるかもしれない
ずーーっと開きっぱなしにしていると時々PC(有線のデスクトップ)でも表示されているのを見かけますが、デスクトップだとそれぐらいかもですね(スリープとかすればそれは流石に切断されると考えられる)
Wifi→モバイルデータやその逆のタイミングなんかでもよく起きるかもしれない
同意します
地下鉄乗ると出やすくていやん
コンフリクト解消
コンフリクト解消(次の次用)
コンフリクト解消
コンフリクト解消