Repository navigation
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
|
…yout # Conflicts: # .vitepress/theme/style.css
Collaborator
Author
|
根拠が持たなかったので閉じる。実装を確かめ直したところ、issue #471 に書いた 2 点とも誤りだった。 「現在地を色だけで示している」は事実と違う。 サイドバーにも右の目次にも 2px のバーが既にある( 技術ブログは「太字+バー」で現在地を示していない。 ブログの実装は「的を持つ部品の現在地は面」で、サイドバーの目次は hover と同じ面( #2893 の射程を広げすぎていた。 「青はレイアウトに使わない」は、読み物の中の導線( 実際に残る不具合は 1 つだけなので、そちらを別に切る。 |
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
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.
close #471
12 件のうち、いちばん見た目が変わる PR。マージ前に見比べてほしい。 VitePress のドキュメントサイトらしい「青い現在地」が無くなるので、良し悪しの判断が要る。
何が問題か
--vp-c-brand-1(#403 で置いた#0a58ca)が、本文の参照リンクの色と、レイアウトの部品の hover / 現在地の両方に使われている。技術ブログ #2893 の「青は読み物の中のリンクの色。レイアウトの導線には使わない」に当たる。もう 1 つ、現在地を色だけで示している。
--vp-c-brand-1と地の文(--vp-c-text-1)の比は 1.70 しかない。技術ブログが #2722 で「色と太さを同時に動かすと現在地だけ浮く」として色を動かさない側に寄せた、その逆の形になっている。入れたもの
hover は文字を一段濃く(
text-2→text-1)、現在地は太さ(700)で示す。もともと持っている印(サイドバーの左のバー・目次のマーカー)だけブランドのネイビーで塗る。新しい面も色も足していない。VPFlyout)text-1/ hover でtext-2(濃さが逆に動く)/ 現在地は青text-2/ hovertext-1/ 現在地text-1+ 700text-1+ 700text-1/ 現在地text-1+ 700 + バーはネイビーtext-1+ 700 + マーカーはネイビーtext-1/ hover で題に下線(枠は動かさない)text-2/ hover でtext-1+ 下線--vp-c-brand-3は暗い側が#0a58caで、地の上で 2.67 と非テキストの 3:1 に届かない(フォーカスリング フォーカスリングとスキップリンクを技術ブログと同じ形にする #437 と同じ向き)VPFeature)は触っていない。hover の枠は.VPFeature.linkにしか当たらず、このサイトの 3 枚はlinkを持たない(押せない)ので、もともと反応しない#appを前置して id で勝たせている静止時の文字の比は
--vp-c-text-2が白地 5.62 / 暗地 5.99、ネイビーのバーは白地 16.24。見比べるところ
documents/の任意のページで、Guidelines… 静止と hover、開いたときのドロップダウンの行判断してほしいのは 1 と 2。 青い現在地は VitePress の通り相場で、読者はそれを「今ここ」と読み慣れている。太さ+ネイビーのバーだけで所在が読めるかを見てほしい。読めないなら、この PR を閉じて「現在地に太さを足すだけ(青は残す)」の狭い版に切り直す。
🤖 Generated with Claude Code