Skip to content
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

Update car image/size #18550

Merged
merged 2 commits into from Nov 7, 2023
Merged

Update car image/size #18550

merged 2 commits into from Nov 7, 2023

Conversation

Redth
Copy link
Member

@Redth Redth commented Nov 6, 2023

Description of Change

In the previous change, the source png was not set to resize=true, and the BaseSize attribute in the project was not set to match the usage in xaml. This make the sizes consistent everywhere, and shrinks the source png a bit, while adding resize=true in order to make the png scale appropriately to different densities.

This should address the regression in startup performance we noticed on Android due to this change.

@Eilon Eilon added the area/templates 📄 Project templates, Item Templates for Blazor and MAUI label Nov 6, 2023
@Redth Redth marked this pull request as ready for review November 6, 2023 22:40
@Redth Redth requested a review from a team as a code owner November 6, 2023 22:40
mattleibow
mattleibow previously approved these changes Nov 6, 2023
@mattleibow mattleibow enabled auto-merge (squash) November 6, 2023 22:42
rmarinho
rmarinho previously approved these changes Nov 7, 2023
The old image name was used in a bunch of tests, it's still the bot so the old name is still fine, and seems like reasonable idea to just revert the name to clean this up.
@Redth Redth dismissed stale reviews from rmarinho and mattleibow via 26f8d73 November 7, 2023 15:28
@PureWeen
Copy link
Member

PureWeen commented Nov 7, 2023

/backport to release/8.0.1xx

Copy link
Contributor

github-actions bot commented Nov 7, 2023

Started backporting to release/8.0.1xx: https://github.com/dotnet/maui/actions/runs/6787960579

@mattleibow mattleibow merged commit e51ded9 into net8.0 Nov 7, 2023
43 of 47 checks passed
@mattleibow mattleibow deleted the dev/redth/template-img-size-fix branch November 7, 2023 17:25
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/templates 📄 Project templates, Item Templates for Blazor and MAUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants