Skip to content

Conversation

@michael-hawker
Copy link
Member

Fixes #3288

Split from #3291 that doesn't update WinUI yet due to issues.

azchohfi and others added 6 commits May 26, 2020 15:07
TODO: depending on width of application it's skipping over items in the Navigation View???
No idea where this got dropped... bad merge?
Fix unused namespaces
@ghost
Copy link

ghost commented May 26, 2020

Thanks michael-hawker for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

@ghost ghost assigned Kyaa-dost May 26, 2020
@ghost ghost added the improvements ✨ label May 26, 2020
@michael-hawker michael-hawker mentioned this pull request May 26, 2020
9 tasks
@michael-hawker michael-hawker added this to the 6.1 milestone May 26, 2020
@michael-hawker michael-hawker requested a review from azchohfi May 27, 2020 22:03
@michael-hawker
Copy link
Member Author

@azchohfi any issues here, basically all the other non-WinUI stuff you already looked at from the other previous PR.

Copy link
Contributor

@azchohfi azchohfi left a comment

Choose a reason for hiding this comment

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

LGTM 🐱‍💻

@michael-hawker michael-hawker merged commit 617a960 into master May 28, 2020
@delete-merged-branch delete-merged-branch bot deleted the mhawker/dependency-update branch May 28, 2020 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update & Consolidate NuGet packages

3 participants