-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Preferences have black text on dark background and are thus not legible in high contrast mode #131
Comments
Could you provide a screenshot please, maybe? 😊 Also, does that only happen in Thunderbird or is it also reproducible in the Firefox version of the extension? |
/cc @tdulcet |
If I chenge TB color settings to Can't say if this is TB bug or by design :) or if there's anything addon can do - but that what I see... |
Also, just in case this be helpful, I have 2 other addons and they both work fine (cardbook and dkim-verifier) |
What do you mean with these overwrite settings? Do you have a support link or a screenshot of the setting, which causes the issue here? |
So some background that may help you to understand what is happening: Without overwrite it should honor So when you overwrite that in your browser via some overwrite, that will of course fail, respectively Thunderbird must somehow honor that.
Also note that this is 100%ly rendered and handled by Thunderbird itself, and the extension only provides the content/text, but Thunderbird does all the rendering and design there. Only the settings itself can be influenced. |
Thanks for doing research / follow up. The option to use OS theme makes no difference - starting to seem like this may well be bug with TB - for sure without that 'override' setting there are parts of thundebird that do NOT respect dark theme which is why it was necessary. Hey if you feel this is really all TB fault feel free to close this one out. Its not critical, as one can always turn off the override - make whatever settings you want and then turn it back to keep TB usable. |
I'll wait for @tdulcet 's reply on that, he has more insight into Thunderbird. Anyway, in the mean time, if you think this is a bug in Thunderbird you may open a bug report for them, respectively search for existing reports, also/or look into the support forum. |
cool thanks! |
A quick check shows that the options page in Thunderbird works as expected with the dark mode without this override: Although I can confirm the issue when enabling the override: However, enabling the same override in Firefox causes the same issue: This override seems to enable the high contrast mode (see here and here) which it not yet well supported by Firefox/Thunderbird (see bug 1016556 and bug 1619744). Although, I checked the options pages for a few other add-ons in Firefox and most did not have this issue, so I would say it is likely a bug with the CSS used by the AutomaticSettings library (it probably needs to be checking prefers-contrast).
Yeah, the fact you need to enable this override at all to make the dark mode work as expected in Thunderbird means that there must be other bugs in Thunderbird. I would encourage you to file bugs on BMO for those. |
Seems its a known bug https://bugzilla.mozilla.org/show_bug.cgi?id=1690531 thanks |
Thanks for all the investigation. I've opened TinyWebEx/AutomaticSettings#23 for it. Also removed the Thunderbird tag as it thus also seems to affect Firefox. |
My knowledge is just about TB extensions, since I ported this add-on and several others to TB, so unfortunately I do not think I would be much help with that bug... Hopefully they will be able to get it fixed for you. |
Bug description
FYI
AEP version 0.9.1
Thunderbird 102.0.1
The preferences tab in addons has black text on very dark/black background.
Drop-downs, radio boxes and a slider are all fine as is the "linked" text at bottom (license file etc).
Linux with dark kde plasma heme and TB general colors set to white text/navy background (override always).
The other sections are legible but not brilliantly so (Details, Permissions and Release notes).
These are faint off white on dark background. Not great but legible.
The text was updated successfully, but these errors were encountered: