Grid: Window management - resize preview does not show when moving a window from the semi-maximised state to the maximised state in a single action

Bug #1071689 reported by John Lea
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ayatana Design
Fix Released
High
John Lea
Compiz
Fix Released
Medium
Sami Jaktholm
0.9.8
Fix Released
Medium
Sami Jaktholm
compiz (Ubuntu)
Fix Released
Medium
Sami Jaktholm
Quantal
Fix Released
Undecided
Timo Jyrinki

Bug Description

[Impact]

Unable to move a window from the semi-maximised state to the maximised state in a single action

[Test Case]

1. Semi-maximise a window
2. In a single drag action, grab the window title bar and drag the window away from the edge (into the restored state), and then with out releasing the left mouse button drag the window to the top of the screen.

[Regression Potential]

Grid plugin behavior in general.

---

Unable to move a window from the semi-maximised state to the maximised state in a single action

To reproduce:

1. Semi-maximise a window
2. In a single drag action, grab the window title bar and drag the window away from the edge (into the restored state), and then with out releasing the left mouse button drag the window to the top of the screen.

What currently incorrectly happens:
- nothing

What should happen:
- the maximise gesture preview should be displayed

Related branches

John Lea (johnlea)
Changed in ayatana-design:
assignee: nobody → John Lea (johnlea)
importance: Undecided → Critical
Changed in compiz:
importance: Undecided → High
Changed in compiz (Ubuntu):
importance: Undecided → High
tags: added: udp
Changed in unity:
importance: Undecided → High
Changed in ayatana-design:
status: New → Triaged
Changed in compiz:
status: New → Triaged
Changed in compiz (Ubuntu):
status: New → Triaged
Changed in unity:
status: New → Triaged
Changed in unity (Ubuntu):
status: New → Triaged
Changed in ayatana-design:
status: Triaged → Fix Committed
Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

It seems like it does actually maximise, but the orange box doesn't appear?

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

It's also not a regression. It's broken in 12.04 too.

Revision history for this message
John Lea (johnlea) wrote :

@popey; humm, perhaps not a regression then.

summary: - [regression] Window management - unable to move a window from the semi-
- maximised state to the maximised state in a single action
+ Window management - unable to move a window from the semi-maximised
+ state to the maximised state in a single action
MC Return (mc-return)
summary: - Window management - unable to move a window from the semi-maximised
- state to the maximised state in a single action
+ Grid: Window management - window preview does not show when moving a
+ window from the semi-maximised state to the maximised state in a single
+ action
MC Return (mc-return)
summary: - Grid: Window management - window preview does not show when moving a
+ Grid: Window management - resize preview does not show when moving a
window from the semi-maximised state to the maximised state in a single
action
MC Return (mc-return)
tags: added: grid precise quantal
Changed in compiz:
milestone: none → 0.9.9.0
Sami Jaktholm (sjakthol)
Changed in compiz:
status: Triaged → In Progress
assignee: nobody → sampo555 (sampo555)
Revision history for this message
MC Return (mc-return) wrote :

The branch created and linked by sampo555 fixes this problem reliably.

sampo555, thanks a lot 4 this fix. :)

no longer affects: unity
no longer affects: unity (Ubuntu)
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Seriously, if we let bugs like this be "Critical" there's no room for classifying bugs that actually are important.

Changed in compiz:
importance: High → Low
Changed in compiz (Ubuntu):
importance: High → Low
Changed in compiz:
status: In Progress → Fix Committed
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Fix committed into lp:compiz at revision 3475, scheduled for release in Compiz 0.9.9.0.

Bumped importance to Medium because I agree this is a usability issue, at least while grid is as sensitive as it is.

Changed in compiz:
importance: Low → Medium
Changed in compiz (Ubuntu):
importance: Low → Medium
John Lea (johnlea)
Changed in ayatana-design:
importance: Critical → High
Stephen M. Webb (bregma)
Changed in compiz (Ubuntu):
milestone: none → ubuntu-13.04-alpha-1
status: Triaged → In Progress
Andrea Azzarone (azzar1)
Changed in compiz (Ubuntu):
status: In Progress → Fix Committed
Changed in compiz (Ubuntu):
assignee: nobody → sampo555 (sampo555)
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.6 KiB)

This bug was fixed in the package compiz - 1:0.9.9~daily12.12.05-0ubuntu1

---------------
compiz (1:0.9.9~daily12.12.05-0ubuntu1) raring; urgency=low

  [ Michael Terry ]
  * debian/control:
    - Update Vcs-Bzr
  * debian/patches/100_expo_layout.patch:
    - Fix to apply
  * Automatic snapshot from revision 3451 (bootstrap)
    - Fix crash from writing to static memory (LP: #1065814)
    - Fix maximized windows changing workspaces (LP: #1071791)
    - Respect Zoom Factor setting (LP: #1066187)
    - Respect Show Window Title setting (LP: #1009999)
    - Restore text background transparency (LP: #1042132)
    - Restore screenshot selection rectangle transparency (LP: #1047788)
    - Restore top and bottom cube transparency (LP: #1048272)
    - Fix scaleaddon window highlighting being solid (LP #1068503)
    - Make resize hint overlay disappear when changing edges (LP: #839602)
    - Preserve grid IDs during drag (LP: #1067812, LP: #1048855)
    - Save current display before doing snap preview (LP: #1068173)
    - Ignore inactive keybindings (LP: #1053280)
    - Handle <primary> in keyboard shortcuts (LP: #936840)
    - Make sure showdesktop plugin is loaded after unityshell (LP: #1067534)
    - Fix grid build failure (LP: #1067577)
    - Fix Jenkins test failures (LP: #1058577)
    - Don't allow duplicate static symbols when linking (LP: #1067964)
    - Make external templates C++0x compatible (LP: #1067598)
    - Make sure to initialize class members (LP: #1033877)
    - Don't duplicate templates in plugins (LP: #1066793, LP: #1066803,
      LP: #1066804, LP: #1066805, LP: #1066799, LP: #1066796,
      LP: #1066795, LP: #1066798, LP: #1067218, LP: #1067219)
    - Don't needlessly set gsettings keys (LP: #1064791)
    - Remove unused code (LP: #1037142, LP: #1067234)
    - Fix syntax highlighting for trunk xml files (LP: #1066823)
    - Fix documentation for building with python2 (LP: #1070211)

  [ Stephen M. Webb ]
  * Windows open below panel and launcher, grid does not work at all,
    window animations missing, and general plugin chaos if built on
    raring (cmake >= 2.8.10) (LP: #1085581)

  [ Iven Hsu ]
  * KDE Appmenu doesn't work with compiz decorators (LP: #1082265)
  * compiz 0.9.8.6 fails to build from source in KDE4-window-decorator
    with latest KDE 4.10 beta (LP: #1082152)
  * kde4-window-decorator crashes when starting up with oxygen theme
    (LP: #1050776)

  [ sampo555 ]
  * Grid: Window management - resize preview does not show when moving a
    window from the semi-maximised state to the maximised state in a
    single action (LP: #1071689)

  [ Timo Jyrinki ]
  * HTML5 video in Firefox continues to tear (LP: #1086337)

  [ MC Return ]
  * Workspacenames Plug-in: Wishlist: Default Appearance values not
    ideal (LP: #1075600)
  * Resizeinfo Plug-in: No possibility to change the color and
    transparency of the rounded background outline (LP: #1070297)
  * Resizeinfo Plug-in: Gradient color 2 and 3 settings are ignored and
    no background gradient is rendered (LP: #1070233)
  * Expo animations "Fade + Zoom" & "Vortex" result in a black screen
    (LP: #875311)
  * Workspacenames Plug-in: Wishlist: No default values...

Read more...

Changed in compiz (Ubuntu):
status: Fix Committed → Fix Released
description: updated
Changed in compiz (Ubuntu Quantal):
status: New → In Progress
assignee: nobody → Timo Jyrinki (timo-jyrinki)
Revision history for this message
Adam Conrad (adconrad) wrote : Please test proposed package

Hello John, or anyone else affected,

Accepted compiz into quantal-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/compiz/1:0.9.8.6-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in compiz (Ubuntu Quantal):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Michael Terry (mterry) wrote :

I can confirm that after the update, Terminal went from failing the test case to passing it (being able to maximize after being semi-maximized in one action).

tags: added: verification-done
removed: verification-needed
Revision history for this message
Scott Kitterman (kitterman) wrote : Update Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.8.6-0ubuntu1

---------------
compiz (1:0.9.8.6-0ubuntu1) quantal-proposed; urgency=low

  * New upstream release.
    - Enable Unredirect Fullscreen Windows by default. It seems to be
      stable enough with all the major graphics drivers now, as at Ubuntu
      12.10 (with updates). (LP: #1063690)
    - Grid: Reset the currently handled window in the GridWindow
      destructor only if the destroyed window was the handled one.
      (LP: #1048855) (LP: #1067812)
    - Grid: Set GridWindow->lastTarget to GridMaximize only if the
      window was fully maximized. (LP: #1071689)
  * Cherry-pick related fixes from upstream:
    - Fixed: unredirected fullscreen windows sliding offscreen were
      staying unredirected (always visible), because compiz thought
      they were fullscreen on a different monitor. Add extra smarts
      to tell the difference between fullscreen and offscreen.
      (LP: #1084401)
    - Add an option to specify which windows are allowed to be unredirected
      when they are fullscreen. Default: Allow unredirection on all window
      types except common video players: Totem, MPlayer, Vlc or Plugin-
      container (Flash or other nested browser windows). (LP: #1051802)
    - Add Firefox to the unredirect exceptions. (LP: #1086337)
 -- Timo Jyrinki <email address hidden> Tue, 04 Dec 2012 12:41:40 +0200

Changed in compiz (Ubuntu Quantal):
status: Fix Committed → Fix Released
Changed in compiz:
status: Fix Committed → Fix Released
John Lea (johnlea)
Changed in ayatana-design:
status: Fix Committed → Fix Released
tags: added: rls-w-incoming
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.