Regression in minimalization of open windows causes click-through in decoration

Bug #1228915 reported by Mateusz Stachowski
32
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Unity
Fix Released
Critical
Christopher Townsend
unity (Ubuntu)
Fix Released
Critical
Christopher Townsend

Bug Description

This bug appeared recently after installation of Unity and Compiz packages that have in it's version this +13.10.20130920-0ubuntu1 numbers.

Steps to reproduce:
1. use CTRL + Super + D shortcut or Alt + Tab and pick show desktop
2. repeat the first step to unminimize windows
3. now try to move window by clicking on it's decoration

Expected behaviour you can drag window around.

This doesn't work instead when I click on the window decoration it becomes inactive (greyed out). When I have two or more windows stacked one upon the other clicking on the decoration results in a switch to window below. In other words it results in a click-through the window decoration.

Related branches

Changed in compiz:
status: New → Confirmed
importance: Undecided → Critical
Changed in compiz (Ubuntu):
status: New → Confirmed
importance: Undecided → Critical
Changed in compiz:
assignee: nobody → Christopher Townsend (townsend)
Changed in compiz (Ubuntu):
assignee: nobody → Christopher Townsend (townsend)
Revision history for this message
Christopher Townsend (townsend) wrote :

After doing some Compiz and Unity debugging, I have that this is a regression in Unity caused by this commit: http://bazaar.launchpad.net/~unity-team/unity/trunk/revision/3515

At this point, I think it's the code change in unityshell.cpp at around line 1340 down to around line 1355. I'll focus in on that for now.

no longer affects: compiz/0.9.10
affects: compiz → unity
affects: compiz (Ubuntu) → unity (Ubuntu)
Changed in unity:
status: Confirmed → In Progress
Changed in unity (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity at revision None, scheduled for release in unity, milestone 7.1.1

Changed in unity:
status: In Progress → Fix Committed
Revision history for this message
ThePhoenix (cosminstefanxp) wrote :

I confirm this still happens in Ubuntu 13.10 Beta 2.

Revision history for this message
Christopher Townsend (townsend) wrote :

@ThePheonix,

The updated Compiz package containing this fix for Saucy has not been released yet. It *should* be released soon though.

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

This bug was fixed in the package unity - 7.1.1+13.10.20130927.1-0ubuntu1

---------------
unity (7.1.1+13.10.20130927.1-0ubuntu1) saucy; urgency=low

  [ Michal Hruby ]
  * Add a property to the Scope class
  * Add parameters to Preview::PerformAction
  * Bump ABI version

  [ Brandon Schaefer ]
  * restart the ibus daemon wit hthe right args for tests.
  * Change sleep from 5 to 10, in hopes of appeasing jenkins.
  * If we are switching view ports tell nux to process the events. Even
    though the viewport switcher does grab the mouse, we will still feed
    nux the events so we can correctly update where the mouse is for the
    launcher. (LP: #1231679)

  [ chris.gagnon ]
  * BlackListDriverError moved to Display. (LP: #1227882)

  [ Marco Trevisan (Treviño) ]
  * UBusMessages: remove useless messages (and relative emission).
  * PanelService: set proper priority for floating position-less NG
    indicators. (LP: #1229460)
  * Unity: Use WindowManager average_color property for getting the
    desktop color Having to deal with UBus here was a mess and caused us
    to re-store the same value too many times. Using a nux property
    gives us all the features we need easily while WindowManager is
    already enough shared to be used for this task.
  * VScrollBarOverlayWindow: add quick fade animation on show/hide.

  [ Chris Townsend ]
  * A regression caused by revno. 3515 breaks the recently fixed
    Quicklist and Tooltip active blur. This fixes it by checking if the
    View is a ViewWindow and if so, use QueueDraw() instead of
    NeedSoftRedraw(). (LP: #1228308)
  * The ShowDesktopHandleWindowInterface teardown was missing it's call
    to DeleteHandler() in a recent commit that refactored the handling
    of some Show Desktop code. (LP: #1228915)

  [ Andrea Azzarone ]
  * Don't use sigc::bind with a static value that should be dynamic.
    (LP: #1228965)

  [ Jeremy Bicha ]
  * recommend telepathy-indicator.

  [ Łukasz 'sil2100' Zemczak ]
  * No new recommends/depends now, since at this point of the cycle it's
    not allowed for this project
  * Re-add the telepathy-indicator recommends as it got dropped from
    Main without any valid reasons

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 3540
 -- Ubuntu daily release <email address hidden> Fri, 27 Sep 2013 12:44:54 +0000

Changed in unity (Ubuntu):
status: In Progress → Fix Released
Andrea Azzarone (azzar1)
Changed in unity:
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.