sumatrapdf icon indicating copy to clipboard operation
sumatrapdf copied to clipboard

Dark and Darker themes ignore TextColor and BackgroundColor

Open NooneAtAll3 opened this issue 1 year ago • 8 comments

SumatraPDF version

  • Version 3.5.1 (or at least the one that reports "you use the latest version" when Check_for_Updates is used)

Describe the bug When using non-Light theme, changing TextColor or BackgroundColor doesn't change the corresponding parts of the document

To Reproduce Steps to reproduce the behavior:

  1. Choose Dark(er) theme
  2. Open Advanced Options
  3. Change BackgroundColor value and save
  4. Observe no change happening

Expected behavior Honestly, I don't know what to expect. I can understand how UI-only control can be expected to change pdf appearance and that "detect non-default values" might give complexity in the future

IMO, ideally, you don't make "Theme" a state. Make it a preset - selecting a theme should overwrite BackgroundColor/TextColor value - allowing further customization from there.
Or maybe add an "overwrite with theme = true" option, so that user won't delete his values accidentally.

Some extreme solution would be to put Themes as customizable objects in the settings - that way both 1) custom themes would be supported, while 2) default ones would stay predictable

Additional context I've been struggling with Sumatra, trying to make everything extra dark for myself ("Black theme" sort of) But right now non-Light themes give green/blue hue to the pdf, while Light theme becomes "Gondor calling for aid" with its white strip on top - situation very unfortunate

Since you already have a mechanism to change UI color based on user input - exposing it in the settings.txt would be a great next step

Thank you for your work

NooneAtAll3 avatar Feb 09 '24 12:02 NooneAtAll3

In an app like this there can not be content theming simply objects rendered in a scale of colours, hence no way to mess with PDF browser colours. What is different about SumatraPDF is it can reverse the range for viewing negatives. However the collateral damage is paper is black and ink is white (NOT theming simply inversion).

Repeatedly this has been seen as a means to fake a theme by altering the end points to light grey or dark grey or any other RGB mix but it is not real body Theming which would require every single pixel or line be edited into another colour. Acrobat Editing Reader is able to alter groups of PDF objects to aid accessibility thus it can use a different model of colour changing.

The User Interface is a different aspect as made of 2 parts. the outer part (Title and ScrollBars) controlled by Windows System and the inner part (tabs and sidebar content) controlled by Visual Studio code. The System colour support in Windows repeatedly changes so from 7 to 8 there was a massive change to "Metro" and it looks like another massive change ? between 10 and 11 ?

When Pseudo "Themes" are added the normal body inversion range needs to be altered by over-ride the user range. Hence the grey on grey. One colour range literally fits all, (pleases some of the people, some of the time but not everybody!) in one theme so a darker range in another theme.

GitHubRulesOK avatar Feb 09 '24 12:02 GitHubRulesOK

I can confirm this issue. It's a bit frsustrating if you want dark backgrounds everywhere: You can get an inverted PDF view with SumatraPDF in a light theme, or you can turn SumatraPDF dark but your document is no longer inverted :-(

In a perfect world, as @NooneAtAll3 suggested, these settings should be per-theme, so we can have a toggle between normal-colored PDF & light SumatraPDF and dark, inverted PDF & dark SumatraPDF direktly in the UI ❤️

fefrei avatar Mar 03 '24 16:03 fefrei

Until pre-release 3.6.15945, custom TextColor/BackgroundColor worked fine in the Dark/Darker theme. Since the bug in the Light theme below was fixed, I've been having issues. https://github.com/sumatrapdfreader/sumatrapdf/issues/4030 It was indeed a bug, but it was a useful bug for Dark/Darker theme users.

pulbitz avatar Mar 04 '24 10:03 pulbitz

image The bookmark tree view font is hard to identified in the darker( or dark ) theme, cause the font color is all most same with the background . please add the settings 'TreeFontTextColor' \ 'TreeFontTextBackgroundColor' \ 'TreeFontTextSelectionColor' for user . thank you!

charygao avatar Mar 06 '24 08:03 charygao

image Also sub-menu options that are not available for any particular instance are very hard to see in Dark and Darker mode. Sumatra v3.6.16355 (Pre-Release) and Win 10 Pro 1803.

Cirkutry avatar Jun 26 '24 16:06 Cirkutry

@Cirkutry that is not a problem they are "greyed out" when inappropriate so with file open and with a favorite available image image

GitHubRulesOK avatar Jun 26 '24 16:06 GitHubRulesOK

@GitHubRulesOK Yes, I understand. What I propose is that when a option is seen but not available to use as you illustrated with the "favorite" feature, the color should have a little more contrast. In most applications unavailable options are greyed out but not to the extent of not being visible.

Cirkutry avatar Aug 06 '24 11:08 Cirkutry

Historically main colour was Yellow and grey not a problem with light theme

image

But a lot of people did not like high contrast so it was muted to grey and then everybody insisted it be darker image

GitHubRulesOK avatar Aug 06 '24 15:08 GitHubRulesOK