Merge lp://qastaging/~azzar1/unity/dnd-enhancements into lp://qastaging/unity

Proposed by Andrea Azzarone
Status: Merged
Merged at revision: 1399
Proposed branch: lp://qastaging/~azzar1/unity/dnd-enhancements
Merge into: lp://qastaging/unity
Diff against target: 1269 lines (+524/-203)
14 files modified
plugins/unityshell/src/AbstractLauncherIcon.h (+4/-2)
plugins/unityshell/src/BamfLauncherIcon.cpp (+99/-69)
plugins/unityshell/src/BamfLauncherIcon.h (+11/-5)
plugins/unityshell/src/DNDCollectionWindow.cpp (+63/-0)
plugins/unityshell/src/DNDCollectionWindow.h (+55/-0)
plugins/unityshell/src/DndData.cpp (+63/-0)
plugins/unityshell/src/DndData.h (+71/-0)
plugins/unityshell/src/Launcher.cpp (+120/-94)
plugins/unityshell/src/Launcher.h (+10/-8)
plugins/unityshell/src/LauncherIcon.cpp (+2/-2)
plugins/unityshell/src/LauncherIcon.h (+10/-8)
plugins/unityshell/src/MockLauncherIcon.h (+2/-2)
plugins/unityshell/src/TrashLauncherIcon.cpp (+11/-11)
plugins/unityshell/src/TrashLauncherIcon.h (+3/-2)
To merge this branch: bzr merge lp://qastaging/~azzar1/unity/dnd-enhancements
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+71003@code.qastaging.launchpad.net

Description of the change

THE DNDCOLLECTION WINDOW
The first objective of this branch is creating a DNDCollectionWindow to get nux/x drag and drop (dnd) data as soon as dnd starts and not when mouse pointer enters launcher window.
The DNDcolletionWindow hides itself as soon as processdndmove is called or when dnd ends (so there should be no "invisble window" problem). After that it emits a collected signal that is handled by the launcher.

THE DNDDATA CLASS
So I have created this facility class to manage the 1-N relationship between uris and uris types. This class uses std::set to avoid data duplication.

C++0x FOR
I have rewrited some for loops using new c++0x for.

To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) wrote :

Im sorry it took me so long to get to this, I was gone last week.

I will be reviewing this with excitement tomorrow! Good work :)

Revision history for this message
Jason Smith (jassmith) wrote :

So after running the branch, it does a great job making the launcher appear and do the hinting as soon as a drag starts, however the actual act of dropping something on the launcher is broken in this branch.

review: Needs Fixing
Revision history for this message
Andrea Azzarone (azzar1) wrote :

So I should fix that?

Revision history for this message
Jason Smith (jassmith) wrote :

Yes

Revision history for this message
Andrea Azzarone (azzar1) wrote :

The problem should be gone away...

Revision history for this message
Jason Smith (jassmith) wrote :

Looking great!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.