-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GraphView] Add drag and drop function [gramps51] #232
base: maintenance/gramps51
Are you sure you want to change the base?
[GraphView] Add drag and drop function [gramps51] #232
Conversation
@vantu5z: Drag&drop does not work for me. Dragging works, symbol is shown, but the dropping to the clipboard fails. |
Matt, can you provide more info? |
Windows 10 There aren't any exception raised. What do you mean with the text fields? |
You can drop to text entry fields (where you can type) and other apps like notepad. |
No drag & drop to text fields is also not working. |
Matt, can you test this: |
This comment has been minimized.
This comment has been minimized.
Neither work on Windows AIO 64bit or 32 bit Gramps 5.1.1 on Win 10 (1903) 64bit |
69aa5fe
to
5bd4638
Compare
@PQYPLZXHGF can you test this on Windows:
|
Now it should work in Windows. |
Confirming new changes work on Windows. @jralls Do you have time to test on MacOS please? |
It sort of works on MacOS. If I control-touch I get a page icon for a drag symbol. Releasing the control key adds a + in a green circle and I can then drag the page icon over to the clipboard window and when I lift my finger from the touchpad it adds the chart's focus person, not the person I dragged. The click-and-drag gesture doesn't work. Attempting to drop on the name field of the filter gramplet does nothing. If I don't release the control key the page icon does a fly-back when I lift my finger from the touchpad. I used the Gramps-5.1.1-1 app bundle with Gtk 3.14.15, copying graphview.py over the latest released one. Unfortunately more recent versions of Gtk are unstable (bugs 9718 and 10434) with Gramps on MacOS though they work fine with C applications. That said, drag and drop work as expected on the regular chart view: click and drag to the clipboard window drops the dragged person with no extra calisthenics required. |
Seems this is touchpad issue. Test in Linux with touchpad it works. Click-touch-and-drag. |
@wroldwiedbwe If you still have this laptop, can you test this PR works for you please? |
f55f693
to
6f2bae1
Compare
rebased |
@sam-m888 I only have my old laptop that has ubuntu 14.04 and gtk 3.10.8. , but yes it seems to work well 👍 |
Been a while, is this still a work in progress? |
We have some problems with MacOS and I don't know how to fix this. On Linux and Windows should work fine. |
6f2bae1
to
73da1c1
Compare
Rebased |
we dont need (drag_source_set) as we use (drag_begin_with_coordinates)
73da1c1
to
aa3748b
Compare
@jralls, I change drag-n-drop activation. Please, can you test if it works for you on Mac now? |
@vantu5z, tested on 5.1.4. Well, click-and-drag now works, and the first drag after startup works as expected except that it changes the focus person to the dragged person. Dropping on another person in the chart brings up an "add relationship" dialog and dropping on the clipboard window adds the person to the clipboard. So far so good. Dropping on the name field in the filter gramplet changes the chart as if I'd double-clicked on the person, which isn't what I'd expect. With the selected person changed by the first drag subsequent drag attempts don't change the selected person and drop the person from the first drag unless I first explicitly change the selected person first by clicking on someone else. To illustrate, suppose I have a chart with Alice, Barbara, Charles, David, and Edward. Charles is the selected person when I switch to Graph view. I click and drag Alice to the clipboard window; Alice becomes the selected person. I click and drag David to the clipboard and get Alice again. I double click on Barbara to make her the selected person, then try again to drag David to the clipboard. That works, and now David is the selected person. I try to drag Charles and get another instance of David unless I click on someone (could be David) first. |
@jralls, it seems you wait alot after click and before move mouse to enable drag. |
@vantu5z That works better. I see that you came to your senses after your comment and changed click recognition to button-up instead of a (very short in human time) timeout on the button down. |
Add drag support from search widget. Now we can find some one and drag it to graph or clipboard. |
fix 11093
I get as base PedigreeView, but we have difference:
PedigreeView have Gtk.Widgets and native drag&drop.
But GraphView use GooCanvas so we have to use workarounds to setup drag&drop.
Features:
Needs to be tested on each of the OS's