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

[Bug] [UWP] Cursor shows "Copy" when DataPackageOperation is set to "None" #12060

Closed
BurkusCat opened this issue Sep 9, 2020 · 0 comments · Fixed by #12160
Closed

[Bug] [UWP] Cursor shows "Copy" when DataPackageOperation is set to "None" #12060

BurkusCat opened this issue Sep 9, 2020 · 0 comments · Fixed by #12160
Assignees
Labels
a/DragAndDrop in-progress This issue has an associated pull request that may resolve it! p/UWP t/bug 🐛
Milestone

Comments

@BurkusCat
Copy link
Contributor

Description

Dragging items on UWP will display the wrong UI for scenarios where you aren't trying to copy data.

Steps to Reproduce

  1. Add a DropGestureRecognizer to a list and a DragGestureRecognizer to items in another list.
  2. On the DropGestureRecognizer, set the DragOver event like this:
        private void DropGestureRecognizer_DragOver(object sender, DragEventArgs e)
        {
            e.AcceptedOperation = DataPackageOperation.None;
        }
  1. On UWP, try dragging an item over to the other list.

Expected Behavior

Copy text and icon is not displayed above the cursor. I'm not sure what the native UWP "look" is for this. If this isn't easily changed dynamically, I'd expect no text and icon to appear even for "Copy" as I believe that is less confusing. It is confusing if "Copy" appears over a drag drop operation that is dragging something to a remove/bin area.

Actual Behavior

A small tooltip with a copy icon and copy text appears over the cursor regardless of the DataPackageOperation.

Basic Information

  • Version with issue: 4.8
  • Last known good version: N/A
  • IDE:
  • Platform Target Frameworks:
    • iOS:
    • Android:
    • UWP: 18362
  • Android Support Library Version:
  • Nuget Packages:
  • Affected Devices:

Screenshots

image

Reproduction Link

Workaround

@BurkusCat BurkusCat added s/unverified New report that has yet to be verified t/bug 🐛 labels Sep 9, 2020
@PureWeen PureWeen self-assigned this Sep 17, 2020
@PureWeen PureWeen removed the s/unverified New report that has yet to be verified label Sep 17, 2020
@PureWeen PureWeen added this to the 5.0.0 milestone Sep 17, 2020
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Sep 18, 2020
rmarinho added a commit that referenced this issue Oct 23, 2020
…#12060

* Fix AcceptedOperation so it propagates to UWP correctly

* - fix issue numbering

* Update Xamarin.Forms.Platform.UAP/VisualElementTracker.cs

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/DragAndDrop in-progress This issue has an associated pull request that may resolve it! p/UWP t/bug 🐛
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants