テーマ設定のUI改善 #4154
テーマ設定のUI改善 #4154
Conversation
|
Preview (prod) → https://4154-prod.traq-preview.trapti.tech/ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4154 +/- ##
=======================================
Coverage 86.35% 86.35%
=======================================
Files 66 66
Lines 4719 4719
Branches 564 564
=======================================
Hits 4075 4075
Misses 638 638
Partials 6 6 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
他のに合わせてSettingsThemeEditとかにしてほしいです:pray:
There was a problem hiding this comment.
あ、すみませんこれファイル名もディレクトリ名も、SettingsThemeEditModalですね(Modalつける)
| </modal-frame> | ||
| </template> | ||
|
|
||
| <script lang="ts"> |
There was a problem hiding this comment.
歴史的経緯により元は分割されてましたが、多分<script lang="ts" setup>の方にまとめちゃって大丈夫です
|
あと、再レビューしてOKな状態になったらここから再レビュー依頼お願いします |
src/components/UI/FormRadio.vue
Outdated
| @@ -1,5 +1,5 @@ | |||
| <template> | |||
| <label :class="$style.label"> | |||
| <label :class="$style.label" :aria-checked="isChecked"> | |||
There was a problem hiding this comment.
あ、ここ:aria-checked="isChecked"消し忘れだと思うので消しておいてほしいです
src/views/Settings/ThemeTab.vue
Outdated
| display: flex; | ||
| align-items: center; | ||
| font-weight: bold; |
There was a problem hiding this comment.
display: flex;
align-items: center;はFormRadio側で定義してるから多分いらなくて、font-weight: bold;は多分全ラジオボタンで共通になるのでFormRadio側に移動してほしいです🙏
src/views/Settings/ThemeTab.vue
Outdated
| .content { | ||
| margin-left: 12px; | ||
| margin-left: 0px; |
#2203
設計図通りにテーマ設定のレイアウト・様式変更してみました。
変更前
変更後
