Skip to content

Remove hover images and replace faulty icon in org.eclipse.ui.console#1837

Merged
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:hoverimages-ui.console
Apr 25, 2025
Merged

Remove hover images and replace faulty icon in org.eclipse.ui.console#1837
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:hoverimages-ui.console

Conversation

@HeikoKlare
Copy link
Contributor

The hover images only replicate the ordinary icons without giving any benefit but just duplicate resources. One of the just added SVGs icons (for wordwrap) is even incorrect (it's different version of the wordwrap icon, so that original and hover icon are slightly different). This change completely removes the hover images in org.eclipse.ui.console, including their code usages and the just provisioned SVGs.

Before:
wordwrap_hover_before

After:
wordwrap_hover_after

The hover images only replicate the ordinary icons without giving any
benefit but just duplicate resources. One of the just added SVGs icons
(for wordwrap) is even incorrect. This change completely removes the
hover images in org.eclipse.ui.console, including their code usages and
the just provisioned SVGs.
@HeikoKlare HeikoKlare marked this pull request as ready for review April 25, 2025 07:53
@github-actions
Copy link
Contributor

Test Results

 1 758 files  ±0   1 758 suites  ±0   1h 24m 45s ⏱️ - 3m 21s
 4 173 tests ±0   4 150 ✅ ±0   23 💤 ±0  0 ❌ ±0 
13 119 runs  ±0  12 952 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit 03de489. ± Comparison against base commit b9d0b9c.

@HeikoKlare
Copy link
Contributor Author

Failing version increment check is for infrastructure reasons and necessary version bump has already been made via #1813

@HeikoKlare HeikoKlare merged commit 26a376c into eclipse-platform:master Apr 25, 2025
10 of 11 checks passed
@HeikoKlare HeikoKlare deleted the hoverimages-ui.console branch April 25, 2025 13:10
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

Successfully merging this pull request may close these issues.

1 participant