notes-extension-gnome icon indicating copy to clipboard operation
notes-extension-gnome copied to clipboard

Handle mouse release even if it happens outside the button it concerns

Open der-ali opened this issue 4 years ago • 5 comments

Peek 2019-08-13 14-28

On GNOME Shell 3.28.4

der-ali avatar Aug 13 '19 12:08 der-ali

You began a click on the note, moved the pointer, and released the click outside of the note before the note followed the pointer, so the note didn't get the release signal. It's still waiting for your click release, and will continue to follow the cursor movement until it gets a release signal.

Edit: I label it "enhancement" because the current behavior is what i expect. A better design might be possible, but here there is no bug it's acting like it should.

maoschanz avatar Aug 13 '19 18:08 maoschanz

I understand, but the problem is when you move the courser a little bit fast, the courser slides out of the note and i need to go back and drag the note again. I think you should fix the note to the mouse-courser as in the folder example Peek 2019-08-15 17-04

der-ali avatar Aug 15 '19 15:08 der-ali

But the folder is managed by the desktop extension, which uses the whole screen (in the background). If i use the whole screen for the notes, it wouldn't be possible to interact with the windows/desktop underneath when the notes are visible

maoschanz avatar Aug 15 '19 15:08 maoschanz

  • [ ] if the note is waiting for motion signals, after [some time, like 300ms] without any signal, move the note to the position of the cursor
  • [ ] same for resize
  • [ ] once moved, if the click is actually released, act like if it had been released on the button
  • [ ] same for resize

downside: using mainloop timeouts is highly discouraged by the (outdated?) blog post explaining how to get the extension approved

maoschanz avatar Jun 07 '20 20:06 maoschanz

I just discovered this helpful extension and that issue appeared. Please consider grabbing the pointer using Gdk.Seat.grab() while moving or resizing notes.

dimdin avatar Sep 14 '21 19:09 dimdin