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

Add Shell Resource file explicitly #12954

Merged
merged 2 commits into from
Nov 27, 2020
Merged

Add Shell Resource file explicitly #12954

merged 2 commits into from
Nov 27, 2020

Conversation

PureWeen
Copy link
Contributor

Description of Change

With the addition of 14393 we setup the ShellStyles resource file to only get included when Min Target is 16299 and removed the reference from the Resources.xaml file. This appears to work fine if you have target framework set to >= 18632 but if you have it set to 17763 or 16299 then it fails to load the resource.

Issues Resolved

Platforms Affected

  • UWP

Testing Procedure

  • Pull down the project on the related issue and test the included nuget

PR Checklist

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

@PureWeen PureWeen added blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. i/regression labels Nov 24, 2020
@PureWeen PureWeen added this to the 5.0.0 milestone Nov 24, 2020
Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine with 5.0.0.6171

@rmarinho rmarinho merged commit 9267efa into 5.0.0 Nov 27, 2020
@rmarinho rmarinho deleted the fix_11505 branch November 27, 2020 13:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/shell 🐚 blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. i/regression p/UWP t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] UWP Shell Crashes in Xamarin Forms 4.7
4 participants