-
Notifications
You must be signed in to change notification settings - Fork 6
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
Comments
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. |
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? |
I'm more a fan of just following whatever |
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: |
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 |
partially addresses #105; but not completely.
## [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)
I have my browser / OS configured for dark mode. The extension is unreadable (white text on white background) in dark mode.
The text was updated successfully, but these errors were encountered: