enhance(client): Renote した時の表示をリップルエフェクトと toast に

[Khsmty]

https://github.com/misskey-dev/misskey/pull/10108#issuecomment-1445343261

apiWithDialog からリップルエフェクトと toast に変更しました。

[codecov[bot]]

Codecov Report

Merging #10116 (d53ee16) into develop (81e6a21) will decrease coverage by 0.01%. The diff coverage is n/a.

:exclamation: Current head d53ee16 differs from pull request most recent head 6e06d0c. Consider uploading reports for the commit 6e06d0c to get more accurate results

@@             Coverage Diff             @@
##           develop   #10116      +/-   ##
===========================================
- Coverage    25.06%   25.05%   -0.01%     
===========================================
  Files          705      705              
  Lines        65151    65167      +16     
  Branches      2307     2307              
===========================================
  Hits         16329    16329              
- Misses       48822    48838      +16     

Impacted Files	Coverage Δ
...ackages/backend/src/server/api/ApiServerService.ts	0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[tamaina]

catchした時の挙動どうしよう

[tamaina]

catchなんてしない気がするからいいか

[Khsmty]

そうですねー、catch しようと思うと結構大変な気がしますねこれ

[syuilo]

サーバーの応答が遅い時にレスポンスが返ってくるまで何も表示されないことになるから解決にならなそう

[syuilo]

いやそうでもないか

[Khsmty]

一応右上でグルグルするので読み込み中であることは分かると思います

[tamaina]

サーバーの応答が遅い場合でもRenoteが完了しているっぽい場合は多いし、むしろその間操作ができないのでとてもしんどい

[Khsmty]

ちなみに試しにこんな感じで rippleeffect の方も実装してみたんですが、皆さん的にはどっちの方がいいですか？ 個人的には toast よりも視覚的には分かりやすいと思うんですが

[tamaina]

p1.a9z.devに適用中

[tamaina]

うーん、個人的にはRenoteできたかどうかを気にしていない（できるものだと思っている）ので邪魔なダイアログが消えてくれればどうでもいいという感想

[tamaina]

https://p1.a9z.dev/notes/9bp47s343h

アンケート実施中

[tamaina]

（15票しかないけど）リップルエフェクト派が優勢っぽい

[syuilo]

toastも欲しいかも

[Khsmty]

toast 追加しましたー

[tamaina]

p1.a9z.devに適用

[tamaina]

toast邪魔だわ

[tamaina]

（stickyTopでいい感じにできないかしら

[Khsmty]

（stickyTopでいい感じにできないかしら

いい感じとはどんな感じでしょうか… もう top に張り付いてはいると思うのですが

[tamaina]

Misskeyアプリ内では、CSS変数の--stickyTopやVueでinject('CURRENT_STICKY_TOP')をtopに適用することで、ヘッダーの下部にsticky topやfixed topを指定することができる

[syuilo]

トースト通知はDOM的にも概念的にも最上位に位置するからstickyTopは違うと思う

[tamaina]

実装がどうとかは置いておいて、モバイル表示でタブにかかるのが微妙だったのでそう言った

[Khsmty]

header のサイズに合わせて 50px 下げてみました

[tamaina]

そんなんでええんかと思ったけど割としっくりきた