ardour/livetrax
13
0
Fork 0
Code

Fix placeholder creation when dragging from one DnDVBox to

Browse Source 
another (should fix #4421).


git-svn-id: svn://localhost/ardour2/branches/3.0@10584 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Carl Hetherington 2011-11-14 14:42:29 +00:00
parent 28cc3197d5
commit 28f328e09c
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
libs/gtkmm2ext/gtkmm2ext/dndvbox.h
View File

@ -416,13 +416,19 @@ private:
/* whether we're in the top or bottom half of the child that we're over */ /* whether we're in the top or bottom half of the child that we're over */
bool top_half = (c - int (c)) < 0.5; bool top_half = (c - int (c)) < 0.5;
if (top_half && (before == _drag_child || at == _drag_child)) { /* Note that when checking on whether to remove a placeholder, we never do
so if _drag_child is 0 as this means that the child being dragged is
coming from a different DnDVBox, so it will never be the same as any
of our children.
*/
if (top_half && _drag_child && (before == _drag_child || at == _drag_child)) {
/* dropping here would have no effect, so remove the visual cue */ /* dropping here would have no effect, so remove the visual cue */
remove_placeholder (); remove_placeholder ();
return false; return false;
} }
if (!top_half && (at == _drag_child || after == _drag_child)) { if (!top_half && _drag_child && (at == _drag_child || after == _drag_child)) {
/* dropping here would have no effect, so remove the visual cue */ /* dropping here would have no effect, so remove the visual cue */
remove_placeholder (); remove_placeholder ();
return false; return false;