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

fix: ライトモード時のplaintextのコードブロックの文字色が薄すぎる問題を修正 #143

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

cp-20
Copy link
Contributor

@cp-20 cp-20 commented Jun 13, 2024

No description provided.

@cp-20 cp-20 requested a review from alter334 June 13, 2024 00:59
Copy link
Contributor

@alter334 alter334 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

単純に質問なのですがこの書き換えで他の言語のコードブロックに影響出たりはしないのでしょうか

@alter334
Copy link
Contributor

また、txtについてだけなぜ極端に暗くなってしまうのかの原因がわかれば教えてくださると幸いです

@cp-20
Copy link
Contributor Author

cp-20 commented Jun 13, 2024

単純に質問なのですがこの書き換えで他の言語のコードブロックに影響出たりはしないのでしょうか

影響はないはずです。デフォルトの文字色を変えただけで、ハイライトはそこから上書きされる形のため

また、txtについてだけなぜ極端に暗くなってしまうのかの原因がわかれば教えてくださると幸いです

何かしらのSyntaxHighlighterが入っていれば勝手にいい感じの色でハイライトしてくれるが、plaintextはデフォルトの文字色 = もともとは薄い色を使うので、薄くなっていたというわけですね

@alter334
Copy link
Contributor

参照している変数名が--vp-c-text-dark-1になっているのは若干非自明かもしれない?
(今は時間ないので仕方ないかもですがもしかしたらもっといい命名をつけれたりでしょうか)

@cp-20
Copy link
Contributor Author

cp-20 commented Jun 13, 2024

参照している変数名が--vp-c-text-dark-1になっているのは若干非自明かもしれない? (今は時間ないので仕方ないかもですがもしかしたらもっといい命名をつけれたりでしょうか)

これはVitePress側が使っている命名だと思います

Copy link
Contributor

@alter334 alter334 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vitepress側なら変数命名は仕方ない感じですか
コードブロック自体が元々ダーク寄りだと考えれば変数名はそれでも仕方ないか というところです

css周りももう少し勉強してちゃんと書ければと思います
修正ありがとうございます

@cp-20 cp-20 merged commit 545bfc4 into main Jun 13, 2024
3 checks passed
@cp-20 cp-20 deleted the fix/lightmode-plaintext-codeblock branch June 13, 2024 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants