Merge lp:~stolowski/unity-2d/dash-always-fullscreen-fix into lp:unity-2d

Proposed by Paweł Stołowski
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 1025
Merged at revision: 1024
Proposed branch: lp:~stolowski/unity-2d/dash-always-fullscreen-fix
Merge into: lp:unity-2d
Diff against target: 33 lines (+14/-1)
2 files modified
shell/dash/Dash.qml (+1/-1)
tests/manual-tests/dash.txt (+13/-0)
To merge this branch: bzr merge lp:~stolowski/unity-2d/dash-always-fullscreen-fix
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
Gerry Boland Pending
Review via email: mp+100150@code.launchpad.net

Commit message

[dash] Properly handle always-fullscreen flag, forcing fullscreen mode on small screens: bind to shellManager.dashAlwaysFullScreen, not shellManager.alwaysFullscreen.

To post a comment you must log in.
Revision history for this message
Albert Astals Cid (aacid) wrote :

Thanks for fixing my typo :-)

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

No commit message specified.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'shell/dash/Dash.qml'
--- shell/dash/Dash.qml 2012-03-20 14:50:42 +0000
+++ shell/dash/Dash.qml 2012-03-30 14:13:26 +0000
@@ -69,7 +69,7 @@
69 Binding {69 Binding {
70 target: shellManager70 target: shellManager
71 property: "dashMode"71 property: "dashMode"
72 value: shellManager.alwaysFullScreen || dash2dConfiguration.fullScreen ?72 value: shellManager.dashAlwaysFullScreen || dash2dConfiguration.fullScreen ?
73 ShellManager.FullScreenMode : ShellManager.DesktopMode73 ShellManager.FullScreenMode : ShellManager.DesktopMode
74 }74 }
7575
7676
=== modified file 'tests/manual-tests/dash.txt'
--- tests/manual-tests/dash.txt 2012-03-29 15:59:53 +0000
+++ tests/manual-tests/dash.txt 2012-03-30 14:13:26 +0000
@@ -86,3 +86,16 @@
86 # * Verify there is no graphic noise left on screen86 # * Verify there is no graphic noise left on screen
8787
88----88----
89 # Test case objectives:
90 # * Check that dash fullscreen mode is forced on small screens
91 # Pre-conditions
92 # * Small screen (anything less than 1280x1084), e.g. 1024x768
93 # * form-factor dconf option set to "desktop"
94 # * full-screen dconf option for dash is unset
95 # Test steps
96 # * Verify dash is not showing
97 # * Press Super
98 # * Verify dash is showing maximized
99 # * Verify that pressing dash maximize button has no effect (dash stays in maximized mode).
100
101----

Subscribers

People subscribed via source and target branches