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

お気に入りチャンネルは未読で別に表示できるように #4321

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Takeno-hito
Copy link
Member

@Takeno-hito Takeno-hito commented Jun 21, 2024

モチベーション

お気に入りチャンネルの未読を別で表示できるようにすることで、ユーザーに柔軟な通知管理を提供できるようにする

スクリーンショット

image

困っているポイント

  • そもそももっと良い実装がありそうなので見て欲しい :kan:
  • 「お気に入り」となっている部分の文言の案が欲しい :kan:
  • DM チャンネルを 「未読」側に寄せるか「お気に入り」側に寄せるか :kan:

@Takeno-hito Takeno-hito added enhancement New feature or request design discussion 実装すべきかどうか、どうやって実装するかを考える必要がある labels Jun 21, 2024
@Takeno-hito Takeno-hito self-assigned this Jun 21, 2024
Copy link

@nokhnaton
Copy link
Contributor

PRありがとうございます!
実装の仕方はいい感じだとです!わかりやすいコードだと思います。

気になった点としては今の表記だと「お気に入り」の中身が「未読のお気に入りチャンネル」を表すものであるのがわかりにくく、また新しい分類を作るほどのものでもないのでは無いかと考えてます。
DMの未読も含めて全部未読を表すものではある以上全部「未読」のタブの中に入れて、どこかに☆マークをつけるなどの方法でお気に入りであることを明記するみたいなのがいいのかなと思いました。

もう一つ、根底からの議論になってしまって申し訳ないのですが、お気に入りの未読が上にあることが本当に嬉しいのかについても少し疑問に思っています。
少なくとも、私は今までお気に入りが上にあって欲しいと思ったことがないので、どういうシチュエーションで上にあって欲しいと思ったかも知りたいです。

@Takeno-hito
Copy link
Member Author

未読を付けているチャンネルが非常に多いときに、その中で重要度が高い / 優先的に見たいチャンネルがあるはず(例えば times と SysAd のような)で、そこを分ける方法が欲しいです。現状未読チャンネルを分ける方法がないので、今ある機能と組み合わせて分けられる方法がお気に入りで、そういう使い方をするのは不自然ではないのでこういう PR になってます〜

@kamij-i
Copy link

kamij-i commented Jun 21, 2024

DMの未読も含めて全部未読を表すものではある以上全部「未読」のタブの中に入れて、どこかに☆マークをつけるなどの方法でお気に入りであることを明記するみたいなのがいいのかなと思いました。

僕も現状の構成をできるだけ維持するなら、それがいいと思ってます。
DM→メンション→未読(お気に入り)→未読(お気に入り以外)

アイコンでの表現、これ以外だとスペース的な余裕がなさそう
image

@nokhnaton
Copy link
Contributor

未読を付けているチャンネルが非常に多いときに、その中で重要度が高い / 優先的に見たいチャンネルがあるはず(例えば times と SysAd のような)で、そこを分ける方法が欲しいです。現状未読チャンネルを分ける方法がないので、今ある機能と組み合わせて分けられる方法がお気に入りで、そういう使い方をするのは不自然ではないのでこういう PR になってます〜

返信感謝です。
通知の重要度で分けられるのは確かにあると便利かもなと思いました。
現状でもチャンネル一覧ののお気に入りタブでもお気に入りかつ未読のチャンネルはわかると思うのですが、それだと開くまでのタップ数が多い(追加で1or2タップ必要)ことや、未読ではないチャンネルが見えること、そもそも存在が分かりづらいなどの理由でで使いづらいって思っているいう認識で大丈夫ですか?

あと連絡の重要度の高低の管理としてはお気に入りかどうかよりも通知か未読管理かで分けている人が多い印象が強いのですが、通知ではなくお気に入りで分けている理由ってありますか?

@Takeno-hito
Copy link
Member Author

現状でもチャンネル一覧ののお気に入りタブでもお気に入りかつ未読のチャンネルはわかると思うのですが、それだと開くまでのタップ数が多い(追加で1or2タップ必要)ことや、未読ではないチャンネルが見えること、そもそも存在が分かりづらいなどの理由でで使いづらいって思っているいう認識で大丈夫ですか?

です!自ら見に行かないといけない(traQ を開いたときに受動的に見れない)ので微妙だと思っています

あと連絡の重要度の高低の管理としてはお気に入りかどうかよりも通知か未読管理かで分けている人が多い印象が強いのですが、通知ではなくお気に入りで分けている理由ってありますか?

これはわからないでもないんですけど、例えば僕は executive/random/chat は通知はつけてない(未読には入れてる)けど、優先して表示されてほしい(お気に入りに追加すると思う)ので、通知よりもお気に入りの方が便利かなぁ、くらいのことを考えてました

@Takeno-hito
Copy link
Member Author

kamij さんの良さそう 👀

お気に入りは若干上に表示されて欲しいと思っているので、
「メンションされている」「お気に入り」「普通の他のチャンネル」、みたいな並び順でチャンネルが並んでても良いです?

@Takeno-hito
Copy link
Member Author

image

変えてみました、どうでしょう?

@Takeno-hito Takeno-hito requested a review from kamij-i June 24, 2024 05:03
@Takeno-hito Takeno-hito marked this pull request as ready for review June 24, 2024 05:03
@kamij-i
Copy link

kamij-i commented Jun 24, 2024

DM→メンション→未読(お気に入り)→未読(お気に入り以外)

お気に入りは若干上に表示されて欲しいと思っているので、
「メンションされている」「お気に入り」「普通の他のチャンネル」、みたいな並び順でチャンネルが並んでても良いです?

って同じ意味だよね?
本当にちょっとだけ思ったのは、⭐️マークが#に挟まれてるのきもいかもなーってとこ
右に⭐️つけると、チャンネル名のテキストボックスが短くなっちゃうからこれしかない可能性もある

@Takeno-hito
Copy link
Member Author

あ、ごめんなさい見落としてました 同じ意味です!

本当にちょっとだけ思ったのは、⭐️マークが#に挟まれてるのきもいかもなーってとこ
右に⭐️つけると、チャンネル名のテキストボックスが短くなっちゃうからこれしかない可能性もある

一旦試してみます

@kamij-i
Copy link

kamij-i commented Jun 24, 2024

助かるな

@Takeno-hito
Copy link
Member Author

Takeno-hito commented Jun 24, 2024

こんな感じです!若干 ☆ がずれてるような気もしますが、これそもそも # の位置が左にずれてしまっていそうなので、# を直したらいい感じになる気がします

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design discussion 実装すべきかどうか、どうやって実装するかを考える必要がある enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants