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
1=== modified file 'shell/dash/Dash.qml'
2--- shell/dash/Dash.qml 2012-03-20 14:50:42 +0000
3+++ shell/dash/Dash.qml 2012-03-30 14:13:26 +0000
4@@ -69,7 +69,7 @@
5 Binding {
6 target: shellManager
7 property: "dashMode"
8- value: shellManager.alwaysFullScreen || dash2dConfiguration.fullScreen ?
9+ value: shellManager.dashAlwaysFullScreen || dash2dConfiguration.fullScreen ?
10 ShellManager.FullScreenMode : ShellManager.DesktopMode
11 }
12
13
14=== modified file 'tests/manual-tests/dash.txt'
15--- tests/manual-tests/dash.txt 2012-03-29 15:59:53 +0000
16+++ tests/manual-tests/dash.txt 2012-03-30 14:13:26 +0000
17@@ -86,3 +86,16 @@
18 # * Verify there is no graphic noise left on screen
19
20 ----
21+ # Test case objectives:
22+ # * Check that dash fullscreen mode is forced on small screens
23+ # Pre-conditions
24+ # * Small screen (anything less than 1280x1084), e.g. 1024x768
25+ # * form-factor dconf option set to "desktop"
26+ # * full-screen dconf option for dash is unset
27+ # Test steps
28+ # * Verify dash is not showing
29+ # * Press Super
30+ # * Verify dash is showing maximized
31+ # * Verify that pressing dash maximize button has no effect (dash stays in maximized mode).
32+
33+----

Subscribers

People subscribed via source and target branches