Skip to content

Add SVGs to org.eclipse.ui #2916

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 2 commits into from
Apr 26, 2025

Conversation

Michael5601
Copy link
Contributor

This PR adds SVGs for all icons in the bundles org.eclipse.ui except for the following as it is not available as SVG yet:

etool16/search.svg

See also this PR for more information.

@Michael5601 Michael5601 force-pushed the org.eclipse.ui.SVGs branch from f63760b to 85d9365 Compare April 15, 2025 12:56
Copy link
Contributor

github-actions bot commented Apr 15, 2025

Test Results

 1 824 files  ±0   1 824 suites  ±0   1h 32m 59s ⏱️ - 1m 57s
 7 918 tests ±0   7 689 ✅  - 1  228 💤 ±0  1 ❌ +1 
23 841 runs  ±0  23 092 ✅  - 1  748 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 24adfa2. ± Comparison against base commit 769ddf9.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare force-pushed the org.eclipse.ui.SVGs branch from 85d9365 to 7f36ae3 Compare April 20, 2025 08:01
Michael5601 and others added 2 commits April 26, 2025 18:06
This commit adds SVGs for all icons in the bundles `org.eclipse.ui` except for the following as it is not available as SVG yet:

etool16/search.svg
Use on-the-fly-generated disabled version of SVG-rasterized icons
instead.
@HeikoKlare HeikoKlare force-pushed the org.eclipse.ui.SVGs branch from 7f36ae3 to 24adfa2 Compare April 26, 2025 16:10
@HeikoKlare
Copy link
Contributor

Failing test is unrelated: #294

@HeikoKlare HeikoKlare merged commit 6e312ce into eclipse-platform:master Apr 26, 2025
9 of 11 checks passed
@Michael5601 Michael5601 deleted the org.eclipse.ui.SVGs branch April 27, 2025 07:58
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.

2 participants