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): チャンネルへのノート投稿導線を改善 #13028

Open
wants to merge 54 commits into
base: develop
Choose a base branch
from

Conversation

mesichicken
Copy link

@mesichicken mesichicken commented Jan 18, 2024

Fix #10762
Fix #12711
Fix #12712
Fix #14549

What

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

290973888-edc2f8b0-270d-411c-891c-e46a2f92afc5.mov

Why

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

Additional info (optional)

#12688
こちらのPRを再度OPENします。

Checklist

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 18, 2024
@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 18, 2024

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

@fruitriin
Copy link
Contributor

おっふ

@mesichicken
Copy link
Author

あら

@fruitriin
Copy link
Contributor

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

@mesichicken
Copy link
Author

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

@kakkokari-gtyih
Copy link
Contributor

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

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

Attention: Patch coverage is 30.89431% with 170 lines in your changes missing coverage. Please review.

Project coverage is 19.31%. Comparing base (dd56623) to head (7e27212).

Files with missing lines Patch % Lines
...ges/frontend/src/components/MkVisibilityPicker.vue 0.00% 116 Missing ⚠️
...ckages/frontend/src/scripts/post-button-handler.ts 0.00% 32 Missing and 1 partial ⚠️
packages/frontend/src/ui/_common_/navbar.vue 0.00% 9 Missing ⚠️
...ges/frontend/src/ui/_common_/navbar-for-mobile.vue 0.00% 8 Missing ⚠️
packages/frontend/src/boot/main-boot.ts 0.00% 2 Missing ⚠️
packages/frontend/src/pages/channel.vue 0.00% 1 Missing ⚠️
packages/frontend/src/router/definition.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13028       +/-   ##
============================================
- Coverage    39.96%   19.31%   -20.65%     
============================================
  Files         1563      729      -834     
  Lines       197744   103976    -93768     
  Branches      3631      992     -2639     
============================================
- Hits         79024    20088    -58936     
+ Misses      118115    83333    -34782     
+ Partials       605      555       -50     

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


🚨 Try these New Features:

@mesichicken mesichicken marked this pull request as ready for review January 18, 2024 07:06
@mesichicken mesichicken changed the title [WIP]開いてるページで投稿先がチャンネルになってほしい 開いてるページで投稿先がチャンネルになってほしい Jan 18, 2024
@syuilo
Copy link
Member

syuilo commented Jan 19, 2024

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

@Sayamame-beans
Copy link
Member

Sayamame-beans commented Jan 19, 2024

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

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

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

@syuilo
Copy link
Member

syuilo commented Jan 19, 2024

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

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

@kakkokari-gtyih
Copy link
Contributor

👀

@kakkokari-gtyih
Copy link
Contributor

コンフリクト解消

@kakkokari-gtyih
Copy link
Contributor

あたらしくふえたfavoritedChannelsCacheを使うようにした

@Sayamame-beans
Copy link
Member

This fix #10762 , #12711 , #12712

@tamaina

This comment was marked as off-topic.

@Sayamame-beans

This comment was marked as off-topic.

@kakkokari-gtyih

This comment was marked as off-topic.

@tai-cha
Copy link
Contributor

tai-cha commented Jul 4, 2024

見る(改善できそうなら手を入れてもいいのかしら)

@tai-cha
Copy link
Contributor

tai-cha commented Jul 4, 2024

/preview 1800

@syuilo
Copy link
Member

syuilo commented Sep 26, 2024

テストがあるとよさそうだわね
(結構複雑に見えるので、テストなしに今後メンテナンスしていくのは難しそう)

@samunohito samunohito removed this from the v2024.11.1? milestone Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment