Skip to content

dark-mode-css-bug#5023

Merged
istio-testing merged 1 commit intoistio:masterfrom
rvennam:dark-command-output
Sep 20, 2019
Merged

dark-mode-css-bug#5023
istio-testing merged 1 commit intoistio:masterfrom
rvennam:dark-command-output

Conversation

@rvennam
Copy link
Copy Markdown
Contributor

@rvennam rvennam commented Sep 19, 2019

Some characters ( <, | ) appear highlighted in dark mode:
image

[ ] Configuration Infrastructure
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@rvennam rvennam requested a review from geeknoid September 19, 2019 14:56
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Sep 19, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 19, 2019
@geeknoid
Copy link
Copy Markdown
Contributor

I've been meaning to track this down for a while.

Instead of fixing the dark theme by changing the light theme, I think it would make sense to change that same property in the _dark-theme-syntax.css. WDYT?

@rvennam
Copy link
Copy Markdown
Contributor Author

rvennam commented Sep 19, 2019

I don't believe this property is needed in light theme because of background color already matches.

@geeknoid
Copy link
Copy Markdown
Contributor

/retest

@istio-testing
Copy link
Copy Markdown
Contributor

In response to a cherrypick label: new pull request created: #5035

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. kind/docs size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants