画面横幅が小さいときのチャンネルヘッダーのレイアウトを変更 #4100
Conversation
|
Preview (prod) → https://4100-prod.traq-preview.trapti.tech/ |
Codecov ReportAll modified lines are covered by tests ✅ 📢 Thoughts on this report? Let us know!. |
|
コードまだ読んでないですが先に2点だけ:pray:
|
|
ありがとうございます、やってみます！ |
There was a problem hiding this comment.
すみません、やっぱりこのファイルの変更は一旦消しておいてほしいです:pray:
多分後で別PRで、トピックをチャンネル名の下に表示するような感じにすると思います(デザインチームともうちょっと話し合うとは思いますが)
https://md.trap.jp/8kLQ2b3BRVOS0yB4OnnC3Q#%E3%83%81%E3%83%A3%E3%83%B3%E3%83%8D%E3%83%AB%E3%83%98%E3%83%83%E3%83%80%E3%83%BC%E3%81%AE%E3%83%88%E3%83%94%E3%83%83%E3%82%AF
There was a problem hiding this comment.
ちなみに、もしトピックを消すとするならここではなくて、このコンポーネントを使っているChannelHeader.vueで<channel-header-topic :class="$style.topic" :channel-id="channelId" />に直接v-ifをつけるような変更になると思います
| <header-tools-item | ||
| :class="$style.notificationIcon" | ||
| :data-state="subscriptionChangeInfo.state" | ||
| :icon-name="subscriptionChangeInfo.iconName" | ||
| :disabled="!subscriptionChangeInfo.canChange" | ||
| :tooltip="subscriptionChangeInfo.tooltip" | ||
| @click="changeToNextSubscriptionLevel" | ||
| /> |
There was a problem hiding this comment.
今と順番が逆になってしまっているので、順番は同じままにしてほしいです！
↓順に今のPC、変更後のPC、変更後のスマホ
|
ありがとうございます！ |
| @@ -50,4 +50,4 @@ const topic = computed(() => channelsMap.value.get(props.channelId)?.topic) | |||
| text-overflow: ellipsis; | |||
| white-space: nowrap; | |||
| } | |||
| </style> | |||
| </style> | |||
There was a problem hiding this comment.
ここファイルの最後の改行が消されちゃっているので、戻しておいてほしいです:pray:
VSCodeの設定で
"files.insertFinalNewline": true
というのを入れると保存時に自動で入れてくれると思います
|
チャンネル名のサイズ後で小さくする前提だったからチャンネル名圧迫されちゃってるのどうしようかな |
|
ありがとうございます！ |
|
改行についてはそれだけ直してcommitしてもらってOKです！ 申し訳ないのですが、改行のcommitだけ一旦してもらったらデザインチームの人と結論出すまでマージは待ってほしいです:pray: |
|
分かりました！ |
・画面横幅が小さい時にも通知設定アイコンが表示されるようにした
・画面横幅が小さい時にトピックが表示されないようにした
close #3975