-
Notifications
You must be signed in to change notification settings - Fork 169
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
Support Windows High Contrast mode #577
Comments
Related ReadiumCSS module: ReadiumCSS-os_a11y.css (it’s more of a placeholder/WIP currently, because media queries are moving at a very quick pace) Question though: does Electron have any ability to detect high-contrast-mode in Windows? (in order to pass a flag to ReadiumCSS) Because currently, there’s And the contrast/forced colors/transparency queries are currently being standardised in Level 5… |
Note you have to use an extension in Chrome for high-contrast: chromium low-vision page |
And related Electron issue: electron/electron#13178 |
Related issue: #83 |
See note about SVG icons: |
In the version of electron we are using, there isn't the event I need to use to detect that the high contrast mode is enable. When do you think you will migrate to electron 6 @danielweck ? |
ASAP. I don't want to leave this for later. Let's tackle the migration early on, so we can run lots of tests. |
Then when the NPM packages are published, we can migrate readium-desktop (but for LCP I will have to recompile the native libs for Windows, Linux, MacOS, 32/64 bits...) |
Ok, cool |
Also note that Electron6 has a known bug with Windows dark mode in developer mode (developer tools not compatible), so as developers, we must switch to light mode when using the dev tools. |
what is the progress status of this issue ? it's narrow linked with the css. It may rather be a dedicated task for the css master. |
Merged issue: #1726 DescriptionUnder Windows, if a high-contrast theme is used, some of the elements in the UI of Thorium Reader are not visible. Examples of such elements are the buttons in the top toolbar and the button with actions for a book. See attached screenshots for examples. Steps to reproduce
Test environment
|
Besides pure-CSS techniques, let's not forget Electron's own system preferences API: https://www.electronjs.org/docs/latest/api/system-preferences#systempreferencesgetcolorcolor-windows-macos |
I don't think we will have time to implement this for Thorium 3.0 but definitely something to explore in 3.1 as a logical follow-up to the introduction of system dark/light modes and additional colour themes |
We cannot do this in Thorium3 but instead in Thorium3.x maybe |
The High Contrast Mode is a system mode, where users can choose background and foreground color, buttons and links, selected text.
Support on this mode has to go through an evolution of Readium CSS.
Seems related to readium/readium-css#6 (themes) and somewhat related to readium/readium-css#65 (issue about highlight).
The text was updated successfully, but these errors were encountered: