Merge lp://qastaging/~azzar1/unity/fix-874380 into lp://qastaging/unity

Proposed by Andrea Azzarone
Status: Merged
Approved by: Andrea Azzarone
Approved revision: no longer in the source branch.
Merged at revision: 1775
Proposed branch: lp://qastaging/~azzar1/unity/fix-874380
Merge into: lp://qastaging/unity
Diff against target: 53 lines (+6/-11)
2 files modified
plugins/unityshell/src/BamfLauncherIcon.cpp (+4/-7)
plugins/unityshell/src/DeviceLauncherIcon.cpp (+2/-4)
To merge this branch: bzr merge lp://qastaging/~azzar1/unity/fix-874380
Reviewer Review Type Date Requested Status
Mirco Müller (community) Approve
Marco Trevisan (Treviño) Approve
Tim Penhey Pending
Review via email: mp+83538@code.qastaging.launchpad.net

Description of the change

Uses a two state item (Lock to launcher/Unlock from launcher) instead of the current checkbox option.

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Good for me.

review: Approve
Revision history for this message
Mirco Müller (macslow) wrote :

I can't compile your branch. This is the error I get:

[ 13%] Building CXX object tests/CMakeFiles/standalone-dash.dir/standalone_dash.cpp.o
In file included from /tmp/fix-874380/tests/standalone_dash.cpp:33:0:
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:83:41: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:86:44: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:88:43: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:90:43: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:93:39: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:99:37: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:103:35: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
/tmp/fix-874380/tests/../plugins/unityshell/src/DashStyle.h:106:36: Fehler: »nux::ButtonVisualState« wurde nicht deklariert
make[2]: *** [tests/CMakeFiles/standalone-dash.dir/standalone_dash.cpp.o] Fehler 1
make[1]: *** [tests/CMakeFiles/standalone-dash.dir/all] Fehler 2
make: *** [all] Fehler 2

review: Needs Fixing
Revision history for this message
Mirco Müller (macslow) wrote :

Ok works now.

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.