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

Unreadable in dark mode #105

Open
ajorg-aws opened this issue May 13, 2022 · 5 comments
Open

Unreadable in dark mode #105

ajorg-aws opened this issue May 13, 2022 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@ajorg-aws
Copy link

I have my browser / OS configured for dark mode. The extension is unreadable (white text on white background) in dark mode.

@alandtse
Copy link
Collaborator

Thanks. We probably need to implement something based on this. https://stackoverflow.com/questions/58880234/toggle-chrome-extension-icon-based-on-light-or-dark-mode-browser

We currently rely on diff match patch's html generation so that may have to be changed.

@alandtse alandtse added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels May 14, 2022
@casasmartinignacio
Copy link

Hi. I want to help with this but it works fine on my computer. I checked this on Windows and Chrome. What is your OS / Browser?

@ajorg-aws
Copy link
Author

I'm more a fan of just following whatever prefers-color-scheme happens to say than I am of implementing any kind of toggle for a specific page or feature.

@alandtse
Copy link
Collaborator

I wasn't suggesting a toggle but just pointing out potential solutions for whoever wants to take a shot at it.

Perhaps a quick edit to the css will resolve this. https://github.com/spdx/spdx-license-diff/tree/master/app/styles. We probably need to edit the css to alter what's coming from dmp though since that's the meat of the display:
https://github.com/spdx/spdx-license-diff/blob/master/app/scripts/worker.js#L300

@ajorg-aws
Copy link
Author

ajorg-aws commented May 16, 2022

Hi. I want to help with this but it works fine on my computer. I checked this on Windows and Chrome. What is your OS / Browser?

Chrome on macOS at the moment, but the important thing is that the page I was viewing was viewed in dark mode.

You should be able to emulate the condition by going to, for example, https://raw.githubusercontent.com/spdx/spdx-license-diff/master/LICENSE and opening the Rendering tab (under More Tools) in the Developer Tools. There you can set prefers-color-scheme: dark at "Emulate CSS media features". There's some documentation about it here: https://developer.chrome.com/docs/devtools/rendering/emulate-css/ but it's admittedly a little tricky to get there.

alandtse added a commit that referenced this issue Jul 29, 2023
partially addresses #105; but not completely.
alandtse pushed a commit that referenced this issue Jul 29, 2023
## [0.9.2](v0.9.1...v0.9.2) (2023-07-29)

### Bug Fixes

* respect dark mode for selection bubble ([b54f528](b54f528)), closes [#105](#105)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants