Unity appears to enable decorations in _MOTIFY_WM_HINTS on its own
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Unity |
Fix Released
|
Medium
|
Marco Trevisan (Treviño) | ||
5.0 |
Fix Released
|
Medium
|
Christopher Townsend | ||
6.0 |
Fix Released
|
Medium
|
Christopher Townsend | ||
unity (Ubuntu) |
Fix Released
|
Medium
|
Christopher Townsend | ||
Precise |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Test Case]
1. Start Steam on Ubuntu
-> Notice that the Steam window has a custom, non-Ubuntu border, window buttons and decorations
2. Maximize the Steam window
3. Minimize the Steam window
4. Un-minimize the Steam window
5. Restore the Steam window
-> Make sure that Steam does not have Ubuntu window decorations, but looks as it looked in step 1.
[Regression Potential]
No real regression potential - in really really impossible conditions is the lack of window decorations for selected, standard windows.
Original description:
A Steam Linux user reported this problem.
He created a video which shows a sequence of minimizes and maximizes. Initially the Steam client window does not have decoration. This is expected as Steam sets _MOTIF_WM_HINTS to disable decorations and never changes them again. As the mins and maxes are performed _MOTIF_WM_HINTS gets changed and decorations are enabled. Checking xprop confirms that the window properties actually are different. As far as I can tell the Steam client is not doing this, so it seems like Unity is?
Here's the movie:
http://
Here's the original GitHub report:
https:/
ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: unity 5.18.0-0ubuntu1
ProcVersionSign
Uname: Linux 3.2.0-36-
NonfreeKernelMo
ApportVersion: 2.0.1-0ubuntu17.1
Architecture: i386
CompizPlugins: [core,composite
Date: Sun Feb 10 14:17:57 2013
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Beta i386 (20120328)
MarkForUpload: True
SourcePackage: unity
UpgradeStatus: No upgrade log present (probably fresh install)
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 unity (Ubuntu): | |
assignee: | Stephen M. Webb (bregma) → Unity Maintainers (unity-maintainers-team) |
status: | New → Confirmed |
Changed in unity: | |
assignee: | nobody → Unity Maintainers (unity-maintainers-team) |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in unity: | |
milestone: | none → 7.0.0 |
status: | Confirmed → Triaged |
Changed in unity: | |
assignee: | Unity Maintainers (unity-maintainers-team) → Christopher Townsend (townsend) |
status: | Triaged → Won't Fix |
status: | Won't Fix → Triaged |
Changed in unity (Ubuntu): | |
status: | Confirmed → Fix Released |
assignee: | Unity Maintainers (unity-maintainers-team) → Christopher Townsend (townsend) |
Changed in unity: | |
status: | Fix Committed → Fix Released |
description: | updated |
description: | updated |
description: | updated |
tags: |
added: verification-done removed: verification-needed |
tags: |
added: verification-done removed: verification-needed |
@bregma This is impacting valve users, could you please take a look? Thanks