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

[net7.0] Fix MauiIcon ForegroundScale on systems with comma as decimal separator #11965

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 8, 2022

Backport of #11690 to net7.0

/cc @Redth @AndreKraemer

Fixes #11685

Specifying the ForegroundScale with a dot caused an issue on systems
with commas as decimal separators. The image got upscaled instead of
downscaled on those systems because the ForegroundScale was parsed
without passing InvariantCulture as culture info. Because of this,
it only worked on systems with a dot as a decimal separator.
@Eilon Eilon added the area-single-project Splash Screen, Multi-Targeting, MauiFont, MauiImage, MauiAsset, Resizetizer label Dec 8, 2022
@Redth Redth enabled auto-merge (squash) December 8, 2022 19:51
@Redth
Copy link
Member

Redth commented Dec 8, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Redth Redth merged commit e0daff2 into net7.0 Dec 8, 2022
@Redth Redth deleted the backport/pr-11690-to-net7.0 branch December 8, 2022 21:53
pictos added a commit to unoplatform/uno.resizetizer that referenced this pull request Mar 24, 2023
Make sure all cultures works.
Solution inspired by: dotnet/maui#11965
@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-single-project Splash Screen, Multi-Targeting, MauiFont, MauiImage, MauiAsset, Resizetizer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants