Skip to content

dark mode , bookmark selection un-readable #4908

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

Open
monowovo opened this issue Apr 27, 2025 · 14 comments
Open

dark mode , bookmark selection un-readable #4908

monowovo opened this issue Apr 27, 2025 · 14 comments

Comments

@monowovo
Copy link

monowovo commented Apr 27, 2025

Is your feature request related to a problem? Please describe.
version: SumatraPDF-prerel , 3.52, win11
abou the bookmark when dark mode as the picture shows:
if you move the mouse pointer in the bookmark area, the content you choose is blue dark, not easy to read.

Image

Describe the solution you'd like
have to modify the color of the shadow.

@GitHubRulesOK
Copy link
Collaborator

That is a very old issue with all such altered inverse colourations but of course depends on the other settings so here we see the hover tootip is the best way to see poor choices of contrasted colour settings

Image

@monowovo
Copy link
Author

monowovo commented May 1, 2025 via email

@GitHubRulesOK
Copy link
Collaborator

Agreed the hover in recent versions is limited to visible short text,
There was a past change in that behaviour when tree view was last changed and since I am not the developer to say why the new variation is more limited will simply leave this issue "open"

@ozone10
Copy link

ozone10 commented May 5, 2025

Default light treeview explorer theme has really poor contrast. To fix it treeview would need to use dark explorer theme on Windows 10 and 11 or handle custom draw to have support for Vista and newer OS.

Here is dark explorer theme applied on both sidebars
Image

@GitHubRulesOK
Copy link
Collaborator

@ozone10
I see your PR targets a couple of UI lines so that is helpful
but this thread was more about flipping selection highlight value to more opposed same as false theming is in effect only using the inverse command.

@ozone10
Copy link

ozone10 commented May 7, 2025

@GitHubRulesOK
that was my point. Default explorer theme for treeview does not allow easy way to change color for selected item. You can change background and text color for "default" state but nothing else.

Using Windows 10 or 11 "dark" explorer theme will fix this issue, also make scrollbars dark and make collapse arrows have better contrast.

@MathisP75
Copy link

@ozone10 Oh yeah, this change is badly needed, I hope it gets merged.

@GitHubRulesOK
Copy link
Collaborator

@MathisP75
The merge will make no real difference, let me show you a collection of current issues and pull requests all in one screenshot.

Top left is where Acrobat Editor can change colours two different ways, on the left the red text on the mouse body can be set to blue by the inbuilt editor functions, which is not possible in other readers which can only alter colour "hinting" as shown top middle.

There are currently two problems with "theming" which is actually control of "hinting" so top right one complaint is controling the relative contrast between the cut off points where the "theming" is interfering with inversion as seen top right compared to lower right.

Another problem with inversion is at bottom left the Current Code page looks comparatively similar to top right but on inversion goes blindingly white with no visible content,

Thus we come to ALL the relevant PRs bottom right and the inversion is "brighter" (looks as if higher contrasts) and the mid bar is absorbed into the blackground but the text is still not visible! and scrollbars do not follow the canvas theming nor does the title bar (but that one alone is possibly my settings).

Image

@MathisP75
Copy link

@GitHubRulesOK I don't know what you are talking about. I was talking about ozone10's fixes for scrollbar color, text color and separator color. Nothing to do with inverting the colors of the page.

@GitHubRulesOK
Copy link
Collaborator

@MathisP75
ALL The PRs are in the lower right so the division is not visible as it is black on black so it works but you cant see where it is.
However the scrollbar is still windows 10 and the text highlighted is black on black so no more visible than before.
These problems are not unique to SumatraPDF as here is same problems in latest Okular the menu bar is invisible on the topbar and the theming does not work with the UI controls in dark on dark (see top left) plus the scrollbar is windows 10!

Image

@MathisP75
Copy link

What kind of messed up version of Okular do you have 😭 it doesn't look like this at all

@GitHubRulesOK
Copy link
Collaborator

@MathisP75
In light mode it looks perfect like this it is windows theming messes it up.

Image

@GitHubRulesOK
Copy link
Collaborator

GitHubRulesOK commented May 23, 2025

@MathisP75
Back to topic in hand on Windows 11 all those PR's
mean we have visible scrollbar with invisible division and still a Windows enforced Top bar and scroll bar but at least inversion is failing but bright. I also added in the suggestion of removing the white bottom border as per your other complaint. The core issue is windows has too many variables without a constant theming system that works in all compilers.

Image

@ozone10
Copy link

ozone10 commented May 23, 2025

I did not do PR for dark scrollbar and treeview style yet, it is in my custom build https://github.com/ozone10/sumatrapdf/tree/DarkMode

Image
Image
Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants