Mir

Merge lp:~alan-griffiths/mir/mir_surface_state_horizmaximized into lp:mir

  1. mir_surface_state_horizmaximized
  2. Merge into development-branch
Proposed by Alan Griffiths
Status: Work in progress
Proposed branch: lp:~alan-griffiths/mir/mir_surface_state_horizmaximized
Merge into: lp:mir
Prerequisite: lp:~vanvugt/mir/hidden
Diff against target: 63 lines (+5/-4)
5 files modified
client-ABI-sha1sums (+1/-1)
common-ABI-sha1sums (+1/-1)
include/common/mir_toolkit/common.h (+1/-0)
platform-ABI-sha1sums (+1/-1)
server-ABI-sha1sums (+1/-1)
To merge this branch: bzr merge lp:~alan-griffiths/mir/mir_surface_state_horizmaximized
Reviewer Review Type Date Requested Status
Alberto Aguirre (community) Approve
Kevin DuBois (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Cemil Azizoglu (community) Approve
Review via email: mp+246311@code.launchpad.net

Commit message

common: add ~alan-griffiths/mir/mir_surface_state_horizmaximized

Description of the change

common: add ~alan-griffiths/mir/mir_surface_state_horizmaximized

There's prior art for this surface state (compiz) and it will simplify some bespoke logic in TilingWindowManager

To post a comment you must log in.
Revision history for this message
Cemil Azizoglu (cemil-azizoglu) wrote :

Ok.

review: Approve
Reply
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:2213
http://jenkins.qa.ubuntu.com/job/mir-ci/2633/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-android-vivid-i386-build/827
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-clang-vivid-amd64-build/827
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-vivid-touch/789
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-vivid-amd64-ci/630
        deb: http://jenkins.qa.ubuntu.com/job/mir-vivid-amd64-ci/630/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/789
        deb: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-builder-vivid-armhf/789/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/mir-mediumtests-runner-mako/3914
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/17041

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/mir-ci/2633/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Kevin DuBois (kdub) wrote :

okay, I guess its a pretty small leap to figure out "horizmaximized" means horizontally maximized.

review: Approve
Reply
Revision history for this message
Alberto Aguirre (albaguirre) wrote :

Sure.

review: Approve
Reply
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I don't see why not...

It's unfortunate we can't leverage the same bitmask trick used in compiz:
   horizmaximixed | vertmaximized == maximized
So you only need to write code for the first two to get all three.

But it's also an advantage in us only having to work with mutually exclusive states.

P.S. In Compiz and regular Ubuntu you horizmaximize by using the right mouse button on the maximize button.

Reply
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

As the prerequisite has got held up in review I've just incorporated this into lp:~alan-griffiths/mir/make-TilingWindowManager-simpler/+merge/246418

Reply

Unmerged revisions

2213. By Alan Griffiths

Add mir_surface_state_horizmaximized

2212. By Alan Griffiths

merge lp:~vanvugt/mir/hidden

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'client-ABI-sha1sums'
2--- client-ABI-sha1sums 2015-01-13 14:48:34 +0000
3+++ client-ABI-sha1sums 2015-01-13 14:48:34 +0000
4@@ -10,7 +10,7 @@
5 b141c4d79802ad626d969249c0004744e5c2a525 include/client/mir_toolkit/mir_wait.h
6 6f7b4ecc22afba923806ed2bd7d8244be90b0cfd include/client/mir_toolkit/version.h
7 3350dac884d6006753de2a955bc7a05663cd9449 include/common/mir_toolkit/client_types.h
8-f698a1953b2685343809cf18c776b5f449ee8516 include/common/mir_toolkit/common.h
9+9c2c41e96c3bd5a163b56960b068292ca3aa0494 include/common/mir_toolkit/common.h
10 fce4c1a9e0d037244f7e9e96ea2d8eaab4fc404c include/common/mir_toolkit/cursors.h
11 f4d39e9893ce6308bddd83a49b90f0051f565323 include/common/mir_toolkit/event.h
12 2507f2929415aa423f9551d3c595c439fe1c6efd include/common/mir_toolkit/events/event_deprecated.h
13
14=== modified file 'common-ABI-sha1sums'
15--- common-ABI-sha1sums 2015-01-13 14:48:34 +0000
16+++ common-ABI-sha1sums 2015-01-13 14:48:34 +0000
17@@ -17,7 +17,7 @@
18 31b9c24e2ce7194aeea6694e81c160354033d28a include/common/mir/optional_value.h
19 9ae8473df05dd9e048a73797f01a2f34f7447554 include/common/mir/time/types.h
20 3350dac884d6006753de2a955bc7a05663cd9449 include/common/mir_toolkit/client_types.h
21-f698a1953b2685343809cf18c776b5f449ee8516 include/common/mir_toolkit/common.h
22+9c2c41e96c3bd5a163b56960b068292ca3aa0494 include/common/mir_toolkit/common.h
23 fce4c1a9e0d037244f7e9e96ea2d8eaab4fc404c include/common/mir_toolkit/cursors.h
24 f4d39e9893ce6308bddd83a49b90f0051f565323 include/common/mir_toolkit/event.h
25 2507f2929415aa423f9551d3c595c439fe1c6efd include/common/mir_toolkit/events/event_deprecated.h
26
27=== modified file 'include/common/mir_toolkit/common.h'
28--- include/common/mir_toolkit/common.h 2015-01-13 14:48:34 +0000
29+++ include/common/mir_toolkit/common.h 2015-01-13 14:48:34 +0000
30@@ -75,6 +75,7 @@
31 differs only in the X coordinate. */
32 mir_surface_state_fullscreen,
33 mir_surface_state_hidden,
34+ mir_surface_state_horizmaximized,
35 mir_surface_states
36 } MirSurfaceState;
37
38
39=== modified file 'platform-ABI-sha1sums'
40--- platform-ABI-sha1sums 2015-01-13 14:48:34 +0000
41+++ platform-ABI-sha1sums 2015-01-13 14:48:34 +0000
42@@ -17,7 +17,7 @@
43 31b9c24e2ce7194aeea6694e81c160354033d28a include/common/mir/optional_value.h
44 9ae8473df05dd9e048a73797f01a2f34f7447554 include/common/mir/time/types.h
45 3350dac884d6006753de2a955bc7a05663cd9449 include/common/mir_toolkit/client_types.h
46-f698a1953b2685343809cf18c776b5f449ee8516 include/common/mir_toolkit/common.h
47+9c2c41e96c3bd5a163b56960b068292ca3aa0494 include/common/mir_toolkit/common.h
48 fce4c1a9e0d037244f7e9e96ea2d8eaab4fc404c include/common/mir_toolkit/cursors.h
49 f4d39e9893ce6308bddd83a49b90f0051f565323 include/common/mir_toolkit/event.h
50 2507f2929415aa423f9551d3c595c439fe1c6efd include/common/mir_toolkit/events/event_deprecated.h
51
52=== modified file 'server-ABI-sha1sums'
53--- server-ABI-sha1sums 2015-01-13 14:48:34 +0000
54+++ server-ABI-sha1sums 2015-01-13 14:48:34 +0000
55@@ -17,7 +17,7 @@
56 31b9c24e2ce7194aeea6694e81c160354033d28a include/common/mir/optional_value.h
57 9ae8473df05dd9e048a73797f01a2f34f7447554 include/common/mir/time/types.h
58 3350dac884d6006753de2a955bc7a05663cd9449 include/common/mir_toolkit/client_types.h
59-f698a1953b2685343809cf18c776b5f449ee8516 include/common/mir_toolkit/common.h
60+9c2c41e96c3bd5a163b56960b068292ca3aa0494 include/common/mir_toolkit/common.h
61 fce4c1a9e0d037244f7e9e96ea2d8eaab4fc404c include/common/mir_toolkit/cursors.h
62 f4d39e9893ce6308bddd83a49b90f0051f565323 include/common/mir_toolkit/event.h
63 2507f2929415aa423f9551d3c595c439fe1c6efd include/common/mir_toolkit/events/event_deprecated.h

Subscribers

People subscribed via source and target branches