モバイルの引き出しで「表示テーマ」の左端がずれているのを直す - #3217
Merged
Merged
Conversation
引き出しの中の「表示テーマ」は、行自身の左右 16px の上に .header-nav-label の 12px が乗って 28px 字下げられていた。下に並ぶ 「カテゴリ」「連載」…は 12px なので、同じ段のラベルが1つだけ右へずれる。 左右の余白は中のラベルに任せる。 あわせて .header-theme の中のアイコンから .svg-icon 既定の margin-right（0.3em = 6px）を戻す。絵しか持たないボタンなので、 残すと箱が 28px から 34px になり、絵が箱の中で左に寄っていた （虫眼鏡・送信ボタンは既に戻している）。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
何を直したか
モバイル（1024px 以下）のヘッダーの引き出しで、先頭の「表示テーマ」の行だけが右へずれていた。
.header-theme-drawerが左右 16px の余白を持ち、その中の.header-nav-labelが自分の 12px を重ねるので、文字の左端が 28px になっていた同じ段のラベルなので、左右の余白は中のラベルに任せて 12px にそろえた。縦の余白は変えていない（行の上端・文字の位置は実測で同じ）。
あわせて
.header-themeの中のアイコンから.svg-icon既定のmargin-right（0.3em = 6px）を戻した。絵しか持たないボタンなので、残ると箱が 28px → 34px になり絵が箱の中で左に寄る。虫眼鏡と送信ボタンは既に同じ理由で戻してある。実測（幅390px・引き出しを開いた状態）
make lint-css/make lint-partialsは通っている。