Skip to content

Add SVGs to org.eclipse.ant.ui #1811

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 24, 2025

Conversation

Michael5601
Copy link
Contributor

This PR adds SVGs for all icons in the bundle org.eclipse.ant.ui except for the following as these are not available as SVG yet:

obj16/eclipse16
wizban/ant_wiz
wizban/export_ant_wiz

See also this PR for more information.

Copy link
Contributor

github-actions bot commented Apr 14, 2025

Test Results

 1 758 files  ±0   1 758 suites  ±0   1h 33m 28s ⏱️ +28s
 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 3d53e1c. ± Comparison against base commit 839289d.

♻️ This comment has been updated with latest results.

@Michael5601 Michael5601 force-pushed the Ant-UI-SVGs branch 2 times, most recently from a498a29 to 2a4f1a7 Compare April 15, 2025 09:47
Michael5601 and others added 2 commits April 24, 2025 19:02
This commit adds SVGs for all icons in the bundle org.eclipse.ant.ui except for the following as these are not available as SVG yet:

obj16/eclipse16
wizban/ant_wiz
wizban/export_ant_wiz
Use on-the-fly-generated disabled version of SVG-rasterized icons
instead.
@HeikoKlare
Copy link
Contributor

Version increment check is failing for infrastructure reasons and no further version bump is required here (already done for 4.36 stream).

@HeikoKlare HeikoKlare merged commit ba714d3 into eclipse-platform:master Apr 24, 2025
10 of 11 checks passed
@Michael5601 Michael5601 deleted the Ant-UI-SVGs branch April 25, 2025 06:46
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