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

Update packages, Fix WPF Sample app #1768

Merged
merged 3 commits into from
Jan 22, 2022
Merged

Update packages, Fix WPF Sample app #1768

merged 3 commits into from
Jan 22, 2022

Conversation

VladislavAntonyuk
Copy link
Contributor

@VladislavAntonyuk VladislavAntonyuk commented Dec 17, 2021

Description of Bug

Update Nuget packages to the latest version, Fix WPF Sample app #606

image

PR Checklist

  • Has a linked Issue, and the Issue has been approved
  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

protected override void OnAppearing()
{
base.OnAppearing();
(BindingContext as BaseGalleryViewModel)?.FilterCommand.Execute(string.Empty);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

hack for WPF sample to fix layout issue

Copy link
Contributor

@brminnick brminnick left a comment

Choose a reason for hiding this comment

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

Thanks Vlad! Let's keep the Xamarin.Forms dependency the same for Xamarin.CommunityToolkit and Xamarin.CommunityToolkit.Marukup 👍

Unless you're saying that there is a WPF bug in the previous version of Xamarin.Forms?

@brminnick
Copy link
Contributor

Ah - got it. So we have to update the Xamarin.Forms dependency in order to fix the WPF bug in the Toolkit.

@VladislavAntonyuk
Copy link
Contributor Author

yes, bug #606

@brminnick brminnick enabled auto-merge (squash) January 22, 2022 02:17
@brminnick brminnick merged commit aa6e2e6 into main Jan 22, 2022
@brminnick brminnick deleted the update-package branch January 22, 2022 02:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants