Regression in minimalization of open windows causes click-through in decoration
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Unity |
Fix Released
|
Critical
|
Christopher Townsend | |
| unity (Ubuntu) |
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.
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
- PS Jenkins bot (community): Approve (continuous-integration) on 2013-09-24
- Marco Trevisan (Treviño): Approve on 2013-09-24
- Brandon Schaefer (community): Approve on 2013-09-23
-
Diff: 25 lines (+3/-2)1 file modifiedplugins/unityshell/src/unityshell.cpp (+3/-2)
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) |
Christopher Townsend (townsend) wrote : | #1 |
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 |
PS Jenkins bot (ps-jenkins) wrote : | #2 |
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 |
ThePhoenix (cosminstefanxp) wrote : | #3 |
I confirm this still happens in Ubuntu 13.10 Beta 2.
Christopher Townsend (townsend) wrote : | #4 |
@ThePheonix,
The updated Compiz package containing this fix for Saucy has not been released yet. It *should* be released soon though.
Launchpad Janitor (janitor) wrote : | #5 |
This bug was fixed in the package unity - 7.1.1+13.
---------------
unity (7.1.1+
[ Michal Hruby ]
* Add a property to the Scope class
* Add parameters to Preview:
* 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 ]
* BlackListDriver
[ 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.
* VScrollBarOverl
[ 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
NeedSoftRed
* The ShowDesktopHand
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-
[ Ł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 |
Changed in unity: | |
status: | Fix Committed → Fix Released |
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.