Bottom-edge window placement doesn't take decoration height into consideration

Bug #1282304 reported by Adam Conrad
48
This bug affects 8 people
Affects Status Importance Assigned to Milestone
Compiz
Fix Released
Medium
Marco Trevisan (Treviño)
Unity
Invalid
Medium
Marco Trevisan (Treviño)
compiz (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

With the new compiz, bottom-edge window placement doesn't take decoration height into account, and thus leads to placing windows slightly off-screen. Easily reproducible by hitting Ctrl-Alt-T four times and noting that the top top-edge terminals are placed correctly, but the two bottom-edge ones are placed just below the bottom of the workspace.

Related branches

Changed in compiz:
status: New → Triaged
Changed in compiz (Ubuntu):
status: New → Triaged
Changed in compiz:
importance: Undecided → Medium
Changed in compiz (Ubuntu):
importance: Undecided → Medium
Changed in compiz:
status: Triaged → Invalid
Changed in compiz (Ubuntu):
status: Triaged → Invalid
Changed in unity:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
milestone: none → 7.2.0
Changed in compiz:
status: Invalid → Triaged
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
milestone: none → 0.9.11.0
Changed in unity:
status: Triaged → Invalid
Changed in compiz:
status: Triaged → In Progress
tags: added: new-decorations
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.11+14.04.20140303-0ubuntu1

---------------
compiz (1:0.9.11+14.04.20140303-0ubuntu1) trusty; urgency=low

  [ Marco Trevisan (Treviño) ]
  * Window: define proper cases when we need to resize a window when
    decorating/undecorating Yeah, again here... So, now when setting the
    window extents, we resize a window only in these cases: 1) We have a
    normal window that has never been decorated (lp:1282304) 2) We're
    (re)decorating a maximized or fullscreen window (lp:1282305) This
    seems to work well in all the cases I tested, with no regressions
    with nedit (that highlighted an issue with old code path) (LP:
    #1282304, #1282305)
  * Opengl, Texture: don't try to create a texture of invalid (empty or
    negative) size This seem to cause also a crash when using software
    rendering. (LP: #1055166)
 -- Ubuntu daily release <email address hidden> Mon, 03 Mar 2014 20:09:55 +0000

Changed in compiz (Ubuntu):
status: Invalid → Fix Released
Changed in compiz:
status: In Progress → Fix Committed
Revision history for this message
Paul (i41bktob-launchpad-net) wrote :

This fix solves the problem, but at the cost of unnecessarily resizing all non-fullscreen windows. Now all new terminals (even those that would not have been placed over the edge of the workspace) are 79 columns by 22 rows, which is ... unsatisfactory.

I have filed bug 1287472 to track this regression.

Revision history for this message
Pojar Geo (geoubuntu) wrote :

Also this introduce introduce bizarre behavior of main window of many applications.

Example with gnome screenshot:

- Then open gnome sreenshot, show up without "Take Screenshot" button which is placed in the bottom of main window. This button becomes visible only when the window is moved. (see video in attachament).

tags: added: regression-update
Revision history for this message
Adam Conrad (adconrad) wrote :

So, this was close, but not quite right. The attached screenshot of my old testcase (hitting Ctrl-Alt-T 4 times) now shows my terminals *almost* hitting the bottom and right edges, but not quite. Still a regression from saucy, though a much less nasty regression than having them off the screen.

Changed in compiz (Ubuntu):
status: Fix Released → Triaged
Revision history for this message
Adam Conrad (adconrad) wrote :

This seems to be correctly fixed with the current compiz (1:0.9.11+14.04.20140310-0ubuntu1)

Changed in compiz (Ubuntu):
status: Triaged → Fix Released
Stephen M. Webb (bregma)
Changed in unity:
milestone: 7.2.0 → none
Stephen M. Webb (bregma)
Changed in compiz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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