Skip to content

Make installer window taller #426

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 3 commits into from
May 28, 2025
Merged

Conversation

mhegazy
Copy link
Contributor

@mhegazy mhegazy commented May 21, 2025

To make room for additional options, this is a short term fix, to make the installer window taller. this is an alternative approach to #423.

The side bar art has solid colors, so extending it works without impacting image quality or aspect ratio.

here is what the result looks like:
image

@lxbndr
Copy link
Contributor

lxbndr commented May 27, 2025

@mhegazy here is 724x2000 SVG version of the original asset: swift_side@4x.svg. Hope this would fit nicely for new requirements. As far as I remember, WiX could use SVG directly, converting it to bitmap underneath. That's why even SVG is upscaled to 4x size (so we get 4x bitmap in the end). I am wondering if new WiX able to bundle SVG as is, rendering it in runtime. Anyway, the asset should be useful.

@mhegazy
Copy link
Contributor Author

mhegazy commented May 27, 2025

thanks @lxbndr ! appreciate it. yeah, the wix installer does not support svg natively so had to convert. i did commit both asserts though to make it easier for future changes

@mhegazy
Copy link
Contributor Author

mhegazy commented May 27, 2025

screenshot in the description updated to match latest changes and new asset.

@compnerd
Copy link
Member

Kicked off a test, can merge once that comes back green.

@compnerd
Copy link
Member

CI passed at https://ci-external.swift.org/job/swift-PR-build-toolchain-windows/5913/

@compnerd compnerd merged commit d12f988 into swiftlang:main May 28, 2025
@mhegazy mhegazy deleted the window-height branch May 28, 2025 16:17
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.

4 participants