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

横幅が小さいときに表示されるのは「☆」(お気に入り設定)よりも「🔔」(通知設定)のがよいのでは #3975

Open
sapphi-red opened this issue May 29, 2023 · 6 comments · May be fixed by #4100
Assignees
Labels
design good first issue Good for newcomers

Comments

@sapphi-red
Copy link
Contributor

https://q.trap.jp/messages/805be7a5-0ba5-41a7-a3df-f1dab1f921c2

@nokhnaton nokhnaton added design good first issue Good for newcomers labels Jul 3, 2023
@damin3A3 damin3A3 self-assigned this Aug 24, 2023
@mehm8128
Copy link
Member

お気に入りボタンを残しておいて、通知ボタンも隣に置くとよさそうという話になりました
お気に入りを置き換えるとなると三点リーダーのメニューに入れないといけなくて、そこのボタンクリックでトグルできるのは違和感があることと、横幅小さいときはチャンネルヘッダーのトピックを完全に消しちゃってよさそうという話になったことが理由です

@damin3A3
Copy link
Contributor

designタグがついていますが、すでにあるアイコンや画面の幅が広い時のレイアウトを使っているので、デザインチームの方には特に相談などいらないでしょうか?

@mehm8128
Copy link
Member

一応↑↑の内容がデザインチームと僕がちょっと話して決まった内容なので、特に追加で相談などはしなくて問題ないです!
横幅が小さいときにトピックをまだ消していないので、そこの順番どうするかは要検討ですが、一旦このままで通知ボタンの追加だけしてPR出してみてほしいです!

@damin3A3
Copy link
Contributor

「横幅が小さいときはチャンネルヘッダーのトピックを完全に消す」もこちらのIssueに含めると勘違いしており、「通知アイコンを常に表示」と「横幅が小さいときはチャンネルヘッダーのトピックを完全に消す」を両方やってコミットしてしまってました!
今から取り消して、通知ボタンの方だけでコミットしなおします、事前に相談しなくてすみません…

@mehm8128
Copy link
Member

あーすみません、僕の書き方が悪かったです:pray:
やってくれたならそのままでいいかもなので、消さずに一旦PR出してもらっても大丈夫です!

@damin3A3
Copy link
Contributor

申し訳ないです、自分がきちんと文章を読めてませんでした…
ありがとうございます、一旦PR出すことにします!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design good first issue Good for newcomers
Projects
None yet
4 participants