Unity

Merge lp:~gordallott/unity/place-activation-on-return into lp:unity

  1. place-activation-on-return
  2. Merge into trunk
Proposed by Gord Allott
Status: Merged
Merged at revision: 1007
Proposed branch: lp:~gordallott/unity/place-activation-on-return
Merge into: lp:unity
Diff against target: 13 lines (+1/-2)
1 file modified
src/PlacesGroupController.cpp (+1/-2)
To merge this branch: bzr merge lp:~gordallott/unity/place-activation-on-return
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+54574@code.launchpad.net

Description of the change

So it turns out place activation on return never actually worked, it was just a fluke of bugs that made it look like it used to work, once those bugs got fixed - it stopped "working".

This makes it actually work ;)

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

that's just, awesome! :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/PlacesGroupController.cpp'
2--- src/PlacesGroupController.cpp 2011-03-17 09:30:07 +0000
3+++ src/PlacesGroupController.cpp 2011-03-23 18:01:42 +0000
4@@ -235,8 +235,7 @@
5
6 if (tile)
7 {
8- nux::Geometry geo = tile->GetGeometry ();
9- tile->OnMouseClick.emit (geo.x, geo.y, 0, 0);
10+ tile->sigClick.emit (tile);
11 return true;
12 }
13 }