Skip to content

Remove hover and duplicate image descriptors in org.eclipse.ui.debug#1839

Merged
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:hover-org.eclipse.debug.ui
Apr 26, 2025
Merged

Remove hover and duplicate image descriptors in org.eclipse.ui.debug#1839
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:hover-org.eclipse.debug.ui

Conversation

@HeikoKlare
Copy link
Copy Markdown
Contributor

In org.eclipse.ui.debug, a bunch of descriptors refer to the same files, thus leading to duplicate resources in JFace's resource manager even though the same image is used. In addition, hover images are explicitly set to the same image as the original image in many cases.

This change removes the unnecessary hover image specifications and de-duplicates the image descriptors to save resources.

@HeikoKlare HeikoKlare force-pushed the hover-org.eclipse.debug.ui branch from 42ba681 to 62337f6 Compare April 25, 2025 12:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2025

Test Results

 1 758 files  ±0   1 758 suites  ±0   1h 25m 58s ⏱️ - 6m 20s
 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 a74956d. ± Comparison against base commit f8e6177.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare force-pushed the hover-org.eclipse.debug.ui branch from 62337f6 to b52fbba Compare April 25, 2025 14:43
@HeikoKlare HeikoKlare force-pushed the hover-org.eclipse.debug.ui branch 2 times, most recently from 8f382a9 to 770c027 Compare April 26, 2025 13:31
In org.eclipse.ui.debug, a bunch of descriptors refer to the same files,
thus leading to duplicate resources in JFace's resource manager even
though the same image is used. In addition, hover images are explicitly
set to the same image as the original image in many cases.

This change removes the unnecessary hover image specifications and
de-duplicates the image descriptors to save resources.
@HeikoKlare HeikoKlare force-pushed the hover-org.eclipse.debug.ui branch from 3dbec0e to a74956d Compare April 26, 2025 13:31
@HeikoKlare HeikoKlare merged commit be4a4aa into eclipse-platform:master Apr 26, 2025
11 checks passed
@HeikoKlare HeikoKlare deleted the hover-org.eclipse.debug.ui branch April 26, 2025 14:05
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