lp:~mc-return/compiz/compiz.merge-fix347390-compiz-cannot-resize-vertically-AND-horizontally-with-keys

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:compiz/0.9.11

Christopher Townsend (community): Disapprove on 2014-05-28

MC Return: Needs Information on 2013-07-25

PS Jenkins bot (community): Approve (continuous-integration) on 2013-07-24

Diff: 12 lines (+1/-1)

1 file modified

plugins/resize/src/logic/src/resize-logic.cpp (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #347390: Compiz can't resize a window vertically AND horizontally with keyboard.

Medium

Fix Released

Link a bug report

Related blueprints

3769. By Matija Skala on 2013-07-21

Fixed keyboard resizing just working for either horizontal or vertical
resizing

(Credits and thanks for this patch, Matija Skala <email address hidden>)

(LP: #347390) !!!

3768. By MC Return on 2013-07-21

src/session.cpp cleanup:

Declaration and assignment of local variables in one line.
(this *should* fix bug #1101405)
Use pre- instead of postfix increments.
Added and removed newlines.
Added missing break (just a style issue).
Fixed indentation.

(LP: #1101405). Fixes: https://bugs.launchpad.net/bugs/1101405.

Approved by PS Jenkins bot, Sam Spilsbury.

3767. By BryanFRitt on 2013-07-21

Fixed the non-working Annotate 'Clear' Button.
Moved this option's CCSM position upwards to keep
the button shortcuts together.

(LP: #1202907). Fixes: https://bugs.launchpad.net/bugs/1202907.

Approved by PS Jenkins bot, MC Return.

3766. By Sami Jaktholm on 2013-07-19

Bias the current viewport in addWindowSizeChanges.

If window is visible in the current viewport, use that viewport when
calculating geometries in addWindowSizeChanges. Otherwise the current method
is used.

This way visible windows are maximized in the current viewport.
However, offscreen windows won't jump back to the active viewport if window
maximizes by itself or addWindowSizeChanges is called for some other reason. Fixes: https://bugs.launchpad.net/bugs/776435.

Approved by PS Jenkins bot, Brandon Schaefer, Sam Spilsbury.

3765. By Sam Spilsbury on 2013-07-19

Adjust acceptance tests for the fix to LP: #1198000

  1. Always set frame extents before updating the frame
  2. Added COMPIZ_NO_CONFIGURE_BUFFER_LOCKS so that we don't have to deal
     with the delayed configure requests logic in the tests where it
     does not help
  3. Split restoredDecorationSize in to restoredDecorationBorderSize
     and restoredDecorationInputSize so that we can measure the size of
     the decoration both in terms of its visible and invisible border
  4. Wait for the default decoration properly
  5. Capture the initial geometry of the window with and without decorations
     when the window gets its first decoration. Use these values to calculate
     the size changes in the window from then on instead of its creation
     values
  6. Just flag when the window was first decorated in setWindowFrameExtents
     rather than checking if it is mapped. We only want to not resize
     the window once.

Approved by PS Jenkins bot, Sam Spilsbury, Andrea Azzarone.

3764. By Sam Spilsbury on 2013-07-18

Remove redundant src/logmessage/include/core/logmessage.h

(LP: #1067246). Fixes: https://bugs.launchpad.net/bugs/1067246.

Approved by PS Jenkins bot, Sam Spilsbury.

3763. By Sam Spilsbury on 2013-07-18

Don't add the frame to the toplevel stack if it hasn't been created yet.

In the event that a window is unreparented or destroyed, we usually need
to add its frame window to the toplevel window stack until the time at
which we recieve a DestroyNotify for it, as there may be incoming
ConfigureNotify events puporting to stack other windows relative to
that frame.

However, this does not apply in the case where we have not yet received
a CreateNotify for the frame window. In that case, it is not possible
for any stacking requests to be made relative to this window, so it
does not need to be added immediately. Instead, we can add it at the
time that we recieve a CreateNotify for it as a regular override
redirect window until the time that it is later destroyed.

(LP: #1171314). Fixes: https://bugs.launchpad.net/bugs/1171314.

Approved by PS Jenkins bot, MC Return.

3762. By Brandon Schaefer on 2013-07-18

If we receive a stateNotifyChange, we check if we are in a max state. If so, overwrite the xwc with either X | W, or Y | H depending on the semi max state with the orig position before we went into a semi max state. This way when we leave any sort of max state the orig position is restored.

If the window is being grabbed we do not overwrite the position, which it gets its own value at that point (instead of the orig pos). Fixes: https://bugs.launchpad.net/bugs/892012.

Approved by PS Jenkins bot, Sam Spilsbury, MC Return, Sami Jaktholm.

3761. By Andrea Azzarone on 2013-07-18

Update window geo only on new decoration size only if mapped.

(LP: #1198000). Fixes: https://bugs.launchpad.net/bugs/1198000.

Approved by PS Jenkins bot, Sam Spilsbury, MC Return.

3760. By Sam Spilsbury on 2013-07-18

Port to new gmock. Disable some faling tests because of weird leaks. Fixes: https://bugs.launchpad.net/bugs/1185265.

Approved by PS Jenkins bot, Brandon Schaefer.