Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Fix UWP threading exception #13883

Closed
wants to merge 2 commits into from
Closed

Conversation

charlesroddie
Copy link

Description of Change

Removes an instance of an inverted dependency where creating a Xamarin.Forms object (including a Label with a font size) causes an exception, leading to crashes on user systems when done off the UI thread.

Issues Resolved

Platforms Affected

  • UWP

Behavioral/Visual Changes

  • UWP apps will not get unexpected crashes when creating Xamarin.Forms objects off the UI thread.
  • There is a rare instance where sizing of fonts could be different (developer has changed ControlContentThemeFontSize manually and then not used font sizes).

Testing Procedure

Force the creation of the following object off the UI thread:

Label(FontSize = 10.)

This should not give an exception.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@rmarinho
Copy link
Member

rmarinho commented Mar 1, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rmarinho rmarinho requested review from PureWeen and hartez March 1, 2021 13:04
@samhouts samhouts added i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often t/bug 🐛 labels Mar 5, 2021
@charlesroddie
Copy link
Author

Review please?

@hartez hartez removed their request for review September 14, 2022 14:53
@jfversluis
Copy link
Member

Now that we're so close to the sunsetting of Xamarin.Forms unfortunately we won't be able to take this in anymore, we're really sorry about that. Nevertheless, thank you so much for your time and effort that you have put into this PR.

Please have a look at the evolution of Xamarin.Forms, .NET MAUI. A lot of development has been going on there. Hopefully this issue was already fixed in that codebase. If not, feel free to port this over to there.

Again, thank you so much for being a contributor and Xamarin.Forms user!

@jfversluis jfversluis closed this Apr 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often p/UWP t/bug 🐛
Projects
None yet
5 participants