Nux

Merge lp:~azzar1/nux/fix-965025 into lp:nux/2.0

Proposed by Andrea Azzarone
Status: Merged
Approved by: Andrea Cimitan
Approved revision: 604
Merged at revision: 605
Proposed branch: lp:~azzar1/nux/fix-965025
Merge into: lp:nux/2.0
Diff against target: 12 lines (+1/-1)
1 file modified
Nux/WindowCompositor.cpp (+1/-1)
To merge this branch: bzr merge lp:~azzar1/nux/fix-965025
Reviewer Review Type Date Requested Status
Jay Taoko (community) Approve
Review via email: mp+99263@code.launchpad.net

Commit message

Fix nux dnd issue with multi-monitor.

Description of the change

== Problem ==
Dnd doesn't work with a multi-monitor setup.

== Fix ==
We should use AbsoluteGeometry in DndEventCycle.

== Test ==
I cannot find a way to auto-test it.

To post a comment you must log in.
Revision history for this message
Jay Taoko (jaytaoko) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Nux/WindowCompositor.cpp'
2--- Nux/WindowCompositor.cpp 2012-03-01 06:11:05 +0000
3+++ Nux/WindowCompositor.cpp 2012-03-26 07:27:34 +0000
4@@ -337,7 +337,7 @@
5 else if (event.type == NUX_DND_DROP)
6 {
7 InputArea* current_dnd_area = GetDnDArea();
8- if (current_dnd_area->GetGeometry().IsPointInside(event.x - event.x_root, event.y - event.y_root))
9+ if (current_dnd_area->GetAbsoluteGeometry().IsPointInside(event.x - event.x_root, event.y - event.y_root))
10 current_dnd_area->HandleDndDrop(event);
11 }
12 }

Subscribers

People subscribed via source and target branches

to all changes: