misskey icon indicating copy to clipboard operation
misskey copied to clipboard

enhance(frontend): チャンネルへのノート投稿導線を改善

Open mesichicken opened this issue 1 year ago • 37 comments

Fix https://github.com/misskey-dev/misskey/issues/10762 Fix https://github.com/misskey-dev/misskey/issues/12711 Fix https://github.com/misskey-dev/misskey/issues/12712

What

開いているページ(mainRouter)がチャンネルのとき、そのチャンネルを投稿先にしたいです

https://github.com/misskey-dev/misskey/assets/57591417/95fd56d1-a265-45ba-8c5a-93b2a7aecf76

Why

投稿ボタンが2つあるため間違えが発生しやすく、NとPのショートカットでチャンネルへ投稿できないため

Additional info (optional)

https://github.com/misskey-dev/misskey/pull/12688 こちらのPRを再度OPENします。

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

mesichicken avatar Jan 18 '24 06:01 mesichicken

@mesichicken 元のやつがOpenになった

kakkokari-gtyih avatar Jan 18 '24 06:01 kakkokari-gtyih

おっふ

fruitriin avatar Jan 18 '24 06:01 fruitriin

あら

mesichicken avatar Jan 18 '24 06:01 mesichicken

なおせちゃったら直しちゃってください!

fruitriin avatar Jan 18 '24 06:01 fruitriin

@kakkokari-gtyih 私のほうで直せるか試してみます。無理そうでしたらcloseします。

mesichicken avatar Jan 18 '24 06:01 mesichicken

元のほうが閉じられたのでこっちベースで進めましょうか

kakkokari-gtyih avatar Jan 18 '24 06:01 kakkokari-gtyih

Codecov Report

Attention: Patch coverage is 31.30081% with 169 lines in your changes missing coverage. Please review.

Project coverage is 20.16%. Comparing base (6dd2e9f) to head (93bca51).

Files Patch % Lines
...ges/frontend/src/components/MkVisibilityPicker.vue 0.00% 116 Missing :warning:
...ckages/frontend/src/scripts/post-button-handler.ts 0.00% 32 Missing and 1 partial :warning:
...ges/frontend/src/ui/_common_/navbar-for-mobile.vue 0.00% 8 Missing :warning:
packages/frontend/src/ui/_common_/navbar.vue 0.00% 8 Missing :warning:
packages/frontend/src/boot/main-boot.ts 0.00% 2 Missing :warning:
packages/frontend/src/pages/channel.vue 0.00% 1 Missing :warning:
packages/frontend/src/router/definition.ts 0.00% 1 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13028      +/-   ##
===========================================
- Coverage    20.37%   20.16%   -0.21%     
===========================================
  Files          698      699       +1     
  Lines        97636    97810     +174     
  Branches      1012     1015       +3     
===========================================
- Hits         19889    19720     -169     
- Misses       77229    77569     +340     
- Partials       518      521       +3     

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

codecov[bot] avatar Jan 18 '24 06:01 codecov[bot]

これだとチャンネルを開いている時にチャンネル外に投稿する手段がなくなりそう?

syuilo avatar Jan 19 '24 23:01 syuilo

これだとチャンネルを開いている時にチャンネル外に投稿する手段がなくなりそう?

デッキUIでないケースでは、それで良いのではないかと思います…? (標準でチャンネル外に投稿出来てしまう挙動で困っていたみたいなので)

一つの選択肢としては、https://github.com/misskey-dev/misskey/pull/12687 も加えて、非デッキUIでのチャンネル表示時はデフォルトの公開範囲がそのチャンネルになるが、公開範囲を変えられるのでチャンネル外にも投稿出来るようにする、とかでしょうか…?

Sayamame-beans avatar Jan 19 '24 23:01 Sayamame-beans

デッキUIでないケースでは、それで良いのではないかと思います…? (標準でチャンネル外に投稿出来てしまう挙動で困っていたみたいなので)

チャンネル開きながらチャンネル外に投稿するシチュエーションは普通にあるわね

syuilo avatar Jan 19 '24 23:01 syuilo

さやまめさんが仰ってる通り、#12687 も合わせればよさそうです? ちょっと導線の形は変わりますが、チャンネルを開きながら外にも投げられるようになります

samunohito avatar Jan 19 '24 23:01 samunohito

(安直かもしれませんが、設定項目生やして今の動作とこのPRの動作を切り替えられるようにするとか…)

samunohito avatar Jan 20 '24 00:01 samunohito

たしかにチャンネル外に登録する手段がなくなるのは問題ですね。 https://github.com/misskey-dev/misskey/pull/12687 も再度オープンするのが良いでしょうか もしくはこのPRに取り込んで1つのPRにしてしまっても良いかもしれません(変更差分的にそんなに大変にならなそうなので)

mesichicken avatar Jan 20 '24 00:01 mesichicken

パスからチャンネル情報を取得するように変更されたことで「チャンネルに投稿」からフォームを開いても色情報が取得されなくなる問題がある デグレの修正が必要

https://github.com/misskey-dev/misskey/assets/57591417/0fc70c9a-ac56-4a35-9a89-aa06e2810eb9

mesichicken avatar Jan 22 '24 03:01 mesichicken

「ノート」ボタンからフォームを開いてもチャンネルの色が表示できるようにしました。

https://github.com/misskey-dev/misskey/assets/57591417/b2765d32-6e08-4cf4-80b3-ba880408da4a

mesichicken avatar Jan 23 '24 01:01 mesichicken

クローズされた https://github.com/misskey-dev/misskey/pull/12687 を本ブランチにマージすると、チャンネル画面からでもチャンネル外に投稿することが可能です。 マージして一つのPRにしてしまったほうが良いでしょうか。

https://github.com/misskey-dev/misskey/assets/57591417/04239924-c6be-4598-a579-fb77a104f2f0

mesichicken avatar Jan 23 '24 01:01 mesichicken

https://github.com/misskey-dev/misskey/pull/12687 を取り込んだうえでUIを若干調整するといいかも

kakkokari-gtyih avatar Jan 23 '24 02:01 kakkokari-gtyih

承知しました!取り込みますね

mesichicken avatar Jan 23 '24 02:01 mesichicken

取り込み完了 UIの調整をする

mesichicken avatar Jan 23 '24 02:01 mesichicken

不自然な挙動をするので修正する ノート投稿後にある条件下でフォームがクリアされない(下書きが残ってしまう?) https://misskey.systems/notes/9ow8esbo88

mesichicken avatar Jan 25 '24 03:01 mesichicken

上記のバグを直した

https://github.com/misskey-dev/misskey/assets/57591417/e5ed777d-104d-4e8d-b812-d0eabeaa5fcd

mesichicken avatar Jan 29 '24 07:01 mesichicken

チャンネルが多くて画面外にはみ出そうなときはきちんとスクロールされるようになっていることを確認 https://github.com/misskey-dev/misskey/assets/57591417/247baba3-01ea-4ecf-a4d2-30852b0d66d9

mesichicken avatar Jan 30 '24 05:01 mesichicken

@kakkokari-gtyih もしUI気になるところありましたらお手すきでご指摘ください🙏

mesichicken avatar Jan 30 '24 05:01 mesichicken

多忙なため2月2日くらいに確認します (動作自体は問題なさそうなので、もうこれでマージしてしまってからこちら側でUIの調整を入れるのでも良さそう)

kakkokari-gtyih avatar Jan 30 '24 09:01 kakkokari-gtyih

作業中

kakkokari-gtyih avatar May 21 '24 08:05 kakkokari-gtyih

done(デザイン・挙動周りの見直しはできたとおもう)

kakkokari-gtyih avatar May 21 '24 10:05 kakkokari-gtyih

CHANGELOGをわすれていた

kakkokari-gtyih avatar May 21 '24 10:05 kakkokari-gtyih

CHANGELOGをわすれていた

Done

kakkokari-gtyih avatar May 21 '24 11:05 kakkokari-gtyih

見る

syuilo avatar May 21 '24 11:05 syuilo

コンフリクト解消

kakkokari-gtyih avatar May 23 '24 08:05 kakkokari-gtyih