Merge lp:~dyams/unity-2d/toggling-dash-maximize into lp:unity-2d

Proposed by Lohith D Shivamurthy
Status: Merged
Approved by: Gerry Boland
Approved revision: 807
Merged at revision: 818
Proposed branch: lp:~dyams/unity-2d/toggling-dash-maximize
Merge into: lp:unity-2d
Diff against target: 12 lines (+1/-1)
1 file modified
panel/applets/appname/windowhelper.cpp (+1/-1)
To merge this branch: bzr merge lp:~dyams/unity-2d/toggling-dash-maximize
Reviewer Review Type Date Requested Status
Tiago Salem Herrmann (community) Approve
Review via email: mp+84204@code.launchpad.net

Description of the change

[Dash] clicking maximize button should toggle the Dash between maximize and unmaximize state

To post a comment you must log in.
Revision history for this message
Lohith D Shivamurthy (dyams) wrote :

Change description in detail:

Now Dash has window buttons (close, minimize & maximize) in the panel.
Maximize button maximizes the dash properly but un-maximize doesn't work.
When the original MR was approved there was not issue like this.
Only after integration i have noticed this problem.
So this patch should solve that dash un-maximize issue.

Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

It works fine.

review: Approve
Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity-2d/17/console reported an error when processing this lp:~dyams/unity-2d/toggling-dash-maximize branch.
Not merging it.

Revision history for this message
Gerry Boland (gerboland) wrote :

Merge failed due to libdee ABI change breaking our tests. Re-approving

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'panel/applets/appname/windowhelper.cpp'
2--- panel/applets/appname/windowhelper.cpp 2011-10-17 12:46:55 +0000
3+++ panel/applets/appname/windowhelper.cpp 2011-12-02 04:26:23 +0000
4@@ -171,7 +171,7 @@
5 wnck_window_get_geometry(d->m_window, &x, &y, &width, &height);
6 const QRect windowGeometry(x, y, width, height);
7 ScreenInfo* screenInfo = ScreenInfo::instance();
8- return screenInfo->availableGeometry() == windowGeometry;
9+ return screenInfo->panelsFreeGeometry() == windowGeometry;
10 } else {
11 return wnck_window_is_maximized(d->m_window);
12 }

Subscribers

People subscribed via source and target branches