Merge lp://staging/~brandontschaefer/unity/lp.966030-fix into lp://staging/unity

Proposed by Brandon Schaefer
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 3380
Proposed branch: lp://staging/~brandontschaefer/unity/lp.966030-fix
Merge into: lp://staging/unity
Diff against target: 143 lines (+50/-3)
3 files modified
plugins/unityshell/src/unityshell.cpp (+24/-3)
unity-shared/PluginAdapter.cpp (+24/-0)
unity-shared/PluginAdapter.h (+2/-0)
To merge this branch: bzr merge lp://staging/~brandontschaefer/unity/lp.966030-fix
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+170180@code.staging.launchpad.net

Commit message

Show desktop now only works for individual workspaces. Also if any new windows are opened, or one is restored on that workspace then the next Show desktop will minimize those. The only time windows are restored are when no new windows have been mapped, and no windows have been restored for that workspace only.

Description of the change

Changes to the show desktop icon:

- Only minimizes windows on current workspace.
- Will only restore windows if and only if:
  -- No new windows have been mapped (opened).
  -- No new windows have been unminimized.
  -- The user has not moved workspaces.
- When we are restoring windows from Show desktop, all of them are unminimzed on all workspaces.

Testing this might be hard, but im looking into it :).

Video does not show the new restore fix, but still good to show what it does now.
Video: http://ubuntuone.com/2G483y5nLc0OBkhHWww1gm

So overall things to manually test:
- Only windows in current workspace get minimized.
- Windows on the same workspace only get restored IFF, no new windows have been opened,
no windows have been restore from that workspace.
- All windows must be restored if returning to a workspace that is empty.
- If any new window, or a window is restore, then the next time a ShowDesktop is activated for that workspace all the windows are minimized (for that workspace only).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Works fine here.

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.