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

Fix AcceptedOperation so it propagates to UWP correctly #12160

Merged
merged 4 commits into from
Oct 23, 2020
Merged

Fix AcceptedOperation so it propagates to UWP correctly #12160

merged 4 commits into from
Oct 23, 2020

Conversation

PureWeen
Copy link
Contributor

@PureWeen PureWeen commented Sep 17, 2020 •
edited
Loading

Description of Change

During a DragOver event UWP wasn't correctly propagating the AcceptedOperation value down to the UWP component. This also adds DragAndDrop UI testing capabilities to UWP

Issues Resolved

Platforms Affected

  • UWP

Testing Procedure

  • UI Test Included

PR Checklist

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

@PureWeen @samhouts
Update Xamarin.Forms.Platform.UAP/VisualElementTracker.cs
c75954c 
Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
@PureWeen PureWeen added the blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. label Oct 7, 2020
@rmarinho rmarinho merged commit 710fd46 into 5.0.0 Oct 23, 2020
@rmarinho rmarinho deleted the fix_12137 branch October 23, 2020 11:56
@samhouts samhouts added this to the 5.0.0 milestone Nov 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hartez hartez hartez approved these changes

@samhouts samhouts samhouts approved these changes

Assignees

@hartez hartez

@samhouts samhouts

Labels
a/DragAndDrop blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. Core p/UWP t/bug 🐛
Projects
None yet
Milestone
5.0.0
Development

Successfully merging this pull request may close these issues.

[Bug] [UWP] Cursor shows "Copy" when DataPackageOperation is set to "None"
4 participants
@PureWeen @hartez @samhouts @rmarinho