Skip to content

[Gtk4] Fix COLOR_LINK_FOREGROUND_RGBA value#3250

Merged
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:linkcolor
Apr 17, 2026
Merged

[Gtk4] Fix COLOR_LINK_FOREGROUND_RGBA value#3250
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:linkcolor

Conversation

@akurtakov
Copy link
Copy Markdown
Member

Use button (instead of link as is on gtk 3) and add "link" css class so correct color can be retrieved.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts how SWT (GTK4) determines the system “link” foreground color so that COLOR_LINK_FOREGROUND_RGBA matches the theme’s actual link styling (notably in themes like HighContrast).

Changes:

  • GTK4: use a temporary GtkButton and add the "link" CSS class before retrieving the link color.
  • GTK3: keep existing behavior using a temporary GtkLabel to retrieve link color.

Comment thread bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Display.java Outdated
Use button (instead of link as is on gtk 3) and add "link" css class so
correct color can be retrieved.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results (linux)

   94 files  ±0     94 suites  ±0   14m 9s ⏱️ +5s
4 577 tests ±0  4 357 ✅ ±0  220 💤 ±0  0 ❌ ±0 
3 335 runs  ±0  3 266 ✅ ±0   69 💤 ±0  0 ❌ ±0 

Results for commit 08a172a. ± Comparison against base commit 20bb38c.

@akurtakov akurtakov merged commit 75a088f into eclipse-platform:master Apr 17, 2026
24 checks passed
@akurtakov akurtakov deleted the linkcolor branch April 17, 2026 10:19
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.

3 participants