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 50 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 31.30081% with 169 lines in your changes missing coverage. Please review.

Project coverage is 19.47%. Comparing base (00ccc22) to head (e9c1a62).
Report is 1 commits behind head on develop.

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 ⚠️
...ges/frontend/src/ui/_common_/navbar-for-mobile.vue 0.00% 8 Missing ⚠️
packages/frontend/src/ui/_common_/navbar.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.75%   19.47%   -20.28%     
============================================
  Files         1541      715      -826     
  Lines       192170   100796    -91374     
  Branches      2545      999     -1546     
============================================
- Hits         76402    19633    -56769     
+ Misses      115206    80617    -34589     
+ Partials       562      546       -16     

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

@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
Collaborator

Sayamame-beans commented Jan 19, 2024

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

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

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

@syuilo
Copy link
Member

syuilo commented Jan 19, 2024

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

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

@kakkokari-gtyih
Copy link
Contributor

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

@kakkokari-gtyih
Copy link
Contributor

CHANGELOGをわすれていた

@kakkokari-gtyih
Copy link
Contributor

CHANGELOGをわすれていた

Done

@syuilo
Copy link
Member

syuilo commented May 21, 2024

見る

@kakkokari-gtyih
Copy link
Contributor

コンフリクト解消

@kakkokari-gtyih
Copy link
Contributor

👀

@kakkokari-gtyih
Copy link
Contributor

コンフリクト解消

@kakkokari-gtyih
Copy link
Contributor

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

@Sayamame-beans
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment