icaclient windows "dancing" when decorated
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Invalid
|
Medium
|
Christopher Townsend | ||
Unity |
Fix Released
|
Medium
|
Christopher Townsend | ||
5.0 |
Fix Released
|
Medium
|
Christopher Townsend | ||
6.0 |
Fix Released
|
Medium
|
Christopher Townsend | ||
compiz (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Precise |
Won't Fix
|
Undecided
|
Unassigned | ||
unity (Ubuntu) |
Fix Released
|
Undecided
|
Christopher Townsend | ||
Precise |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Test Case]
1. Download testcase.py (attached to comment no. 3)
2. Run with `python testcase.py`
3. Wait for the window to restore itself
-> Make sure there are no decorations on the restored window
[Regression Potential]
No real regression potential - in really really impossible conditions is the lack of window decorations for selected, standard windows.
Original description:
ICAclient applications, ie reomte windows applications displayed by the Citrix icaclient program display a strange behavior when running with Unity 12.10. When the windows are re-parented and decorated with a border, they move by themselves like they were "dancing" around window moves.
Pre-requisites:
- install the icaclient packages from http://
- create an account on the Citrix Cloud demo site: http://
To reproduce:
1. open the Financial Dashboards (Excel), at http://
2. If the application starts in windowed mode, but undecorated, do maximize it with the windows controls at the top right of the window
3. Once maximized, un-maximize the window with the Unity controls at the top left on the menu bar
4. Verify that the window is now un-maximized *and* decorated with a Unity border
5. Grab the window by its decoration and move it around in successive drag and drops
Expected result:
- the window should move and stay at the position where it has been dropped
What happens instead:
- the window starts dancing and oscillate around the position where it has been dropped
Ref: https:/
Related branches
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Andrea Azzarone (community): Approve
- John Lea (community): Approve (design)
- Thomi Richards: Pending requested
-
Diff: 3056 lines (+1103/-457)26 files modifiedlauncher/CMakeLists.txt (+0/-1)
launcher/Launcher.cpp (+1/-1)
launcher/StandaloneSwitcher.cpp (+3/-1)
launcher/SwitcherController.cpp (+3/-3)
launcher/SwitcherController.h (+1/-1)
launcher/SwitcherView.cpp (+82/-120)
launcher/SwitcherView.h (+8/-9)
panel/PanelMenuView.cpp (+3/-11)
plugins/unityshell/src/unityshell.cpp (+188/-122)
plugins/unityshell/src/unityshell.h (+10/-3)
plugins/unityshell/src/unityshell_glow.cpp (+3/-3)
tests/CMakeLists.txt (+2/-0)
tests/autopilot/unity/emulators/screen.py (+8/-0)
tests/autopilot/unity/emulators/switcher.py (+11/-1)
tests/autopilot/unity/tests/test_switcher.py (+31/-8)
tests/test_layout_system.cpp (+158/-0)
tests/test_main.cpp (+0/-2)
unity-shared/AbstractIconRenderer.h (+1/-2)
unity-shared/CMakeLists.txt (+1/-0)
unity-shared/LayoutSystem.cpp (+57/-58)
unity-shared/LayoutSystem.h (+19/-22)
unity-shared/PluginAdapter.cpp (+216/-73)
unity-shared/PluginAdapter.h (+8/-1)
unity-shared/StandaloneWindowManager.cpp (+231/-15)
unity-shared/StandaloneWindowManager.h (+45/-0)
unity-shared/WindowManager.h (+13/-0)
- Marco Trevisan (Treviño): Approve
-
Diff: 342 lines (+113/-35)6 files modifiedpanel/PanelMenuView.cpp (+3/-3)
unity-shared/PluginAdapter.h (+5/-2)
unity-shared/PluginAdapterCompiz.cpp (+93/-30)
unity-shared/PluginAdapterStandalone.cpp (+6/-0)
unity-shared/WindowManager.cpp (+5/-0)
unity-shared/WindowManager.h (+1/-0)
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Brandon Schaefer (community): Approve
-
Diff: 318 lines (+106/-34)5 files modifiedplugins/unityshell/src/PanelMenuView.cpp (+3/-3)
plugins/unityshell/src/PluginAdapter.cpp (+91/-29)
plugins/unityshell/src/PluginAdapter.h (+6/-2)
plugins/unityshell/src/WindowManager.cpp (+5/-0)
plugins/unityshell/src/WindowManager.h (+1/-0)
- Didier Roche-Tolomelli: Approve
-
Diff: 762 lines (+265/-104)16 files modifiedAUTHORS (+1/-0)
CMakeLists.txt (+1/-1)
ChangeLog (+72/-0)
debian/changelog (+17/-0)
manual-tests/WindowButtons.txt (+14/-0)
plugins/unityshell/src/LauncherController.cpp (+1/-1)
plugins/unityshell/src/PanelMenuView.cpp (+3/-3)
plugins/unityshell/src/PluginAdapter.cpp (+91/-29)
plugins/unityshell/src/PluginAdapter.h (+6/-2)
plugins/unityshell/src/ResultRendererTile.cpp (+26/-24)
plugins/unityshell/src/WindowManager.cpp (+5/-0)
plugins/unityshell/src/WindowManager.h (+1/-0)
plugins/unityshell/src/unityshell.cpp (+16/-28)
plugins/unityshell/src/unityshell.h (+1/-1)
tests/test_launcher_controller.cpp (+10/-0)
tests/test_service_lens.c (+0/-15)
Changed in compiz: | |
assignee: | nobody → David Barth (dbarth) |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in compiz: | |
assignee: | David Barth (dbarth) → Christopher Townsend (townsend) |
Changed in compiz: | |
status: | Confirmed → Invalid |
Changed in compiz (Ubuntu): | |
status: | Confirmed → Invalid |
Changed in unity: | |
status: | New → Confirmed |
Changed in unity (Ubuntu): | |
status: | New → Confirmed |
Changed in unity: | |
importance: | Undecided → Medium |
assignee: | nobody → Christopher Townsend (townsend) |
Changed in unity: | |
milestone: | none → 7.0.0 |
status: | Confirmed → Fix Committed |
Changed in unity (Ubuntu): | |
assignee: | nobody → Christopher Townsend (townsend) |
Changed in unity (Ubuntu): | |
status: | Confirmed → Fix Released |
Changed in unity: | |
status: | Fix Committed → Fix Released |
description: | updated |
description: | updated |
tags: |
added: verification-done removed: verification-needed |
xev log of the xevents observed during the "dance"