Skip to content

[GTK] Implement Display.getHighContrast #2372

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

Merged
merged 1 commit into from
Aug 4, 2025

Conversation

akurtakov
Copy link
Member

Gtk ships with "HighContrast" and "HighContrastInverse" and Mate desktop ships "ContrastHigh*" theme. If the gtk theme is any of these say it's "high contrast" theme.

Gtk ships with "HighContrast" and "HighContrastInverse" and Mate desktop
ships "ContrastHigh*" theme. If the gtk theme is any of these say it's
"high contrast" theme.
Copy link
Contributor

github-actions bot commented Aug 4, 2025

Test Results

   546 files  ±0     546 suites  ±0   28m 8s ⏱️ - 7m 8s
 4 425 tests ±0   4 408 ✅ ±0   17 💤 ±0  0 ❌ ±0 
16 746 runs  ±0  16 619 ✅ ±0  127 💤 ±0  0 ❌ ±0 

Results for commit 77adb6d. ± Comparison against base commit 007426f.

@merks
Copy link
Contributor

merks commented Aug 4, 2025

Maybe a dumb thought. But are there low contrast themes? Maybe one could just check for contrast and assume high.

@akurtakov
Copy link
Member Author

akurtakov commented Aug 4, 2025

If GTK and theme developers claim only a few of these are "HighContrast" I would trust them rather than provide new meaning of the same terms.
No one knows what weird names one might have come up with so "Contrast" is not enough for me.

@akurtakov
Copy link
Member Author

I have searched a bit and there seem to be efforts for "low contrast" themes https://www.gnome-look.org/p/1013697 . It's in the description but nothing guarantees that no one named a theme "LowerContrast"/"LessContrast" or anything similar.

@akurtakov
Copy link
Member Author

As per previous comments I'm merging this one. It's better to have 100% sure API than "guesses".

@akurtakov akurtakov merged commit 9baef58 into eclipse-platform:master Aug 4, 2025
17 checks passed
@github-project-automation github-project-automation bot moved this to Done in SWT work Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants