clicking on top panel should raise maximized windows
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Unity |
Fix Released
|
Undecided
|
Unassigned | ||
| unity-2d |
Invalid
|
Medium
|
Unassigned | ||
| unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Precise |
Fix Released
|
Undecided
|
Unassigned | ||
| unity-2d (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
| Precise |
Invalid
|
Undecided
|
Unassigned | ||
Bug Description
[Test Case]
1. Start a maximized application
2. Start a unmaximized (restored) application
3. Focus the unmaximized (restored) application
4. Left-click the panel titlebar
-> The maximized application should get focused and moved to the top
[Regression Potential]
Potentially clicking the panel with the mouse button can stop working or work incorrectly.
Original description:
This bug is very similar to bug #703411 and should be able to be fixed in much the same way. This issue comes up when raise-on-click is deactivated, in which case it is impossible to raise a maximized window that is not already on top. Clicking on the top panel should raise the window, just like middle-clicking lowers it. The fix should be reasonably simple; please let me know if I should provide a patch.
Related branches
- Neil J. Patel (community): Approve
- Jason Smith (community): Approve
-
Diff: 73 lines (+19/-0)5 files modifiedsrc/PanelMenuView.cpp (+1/-0)
src/PluginAdapter.cpp (+11/-0)
src/PluginAdapter.h (+1/-0)
src/WindowManager.cpp (+5/-0)
src/WindowManager.h (+1/-0)
- Gerry Boland: Pending requested
-
Diff: 98 lines (+53/-1)4 files modifiedpanel/applets/appname/appnameapplet.cpp (+6/-0)
panel/applets/appname/windowhelper.cpp (+26/-0)
panel/applets/appname/windowhelper.h (+2/-0)
tests/manual-tests/panel.txt (+19/-1)
| Jorge Castro (jorge) wrote | #1 |
| Changed in unity: | |
| status: | New → Confirmed |
| tags: | added: bitesize |
| Tom Jaeger (thjaeger) wrote | #2 |
| Changed in unity-2d: | |
| status: | New → Confirmed |
| Changed in unity (Ubuntu): | |
| status: | New → Confirmed |
| Changed in unity-2d: | |
| importance: | Undecided → Medium |
| tags: | added: delta-with-3d |
| Omer Akram (om26er) wrote | #3 |
| Changed in unity: | |
| status: | Confirmed → Fix Released |
| Changed in unity (Ubuntu): | |
| status: | Confirmed → Fix Released |
| Changed in unity-2d (Ubuntu): | |
| status: | New → Confirmed |
| Changed in unity-2d: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Albert Astals Cid (aacid) |
| milestone: | none → 5.14 |
| Changed in unity-2d: | |
| status: | In Progress → Fix Committed |
| description: | updated |
| Changed in unity (Ubuntu Precise): | |
| status: | New → Fix Released |
| Changed in unity-2d: | |
| milestone: | 5.14 → 5.12.1 |
| description: | updated |
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in unity-2d (Ubuntu Precise): | |
| status: | New → Confirmed |
| Changed in unity-2d: | |
| milestone: | 5.12.1 → 5.14 |
| Changed in unity-2d: | |
| milestone: | 5.14 → none |
| status: | Fix Committed → In Progress |
| Albert Astals Cid (aacid) wrote | #5 |
| Changed in unity-2d: | |
| assignee: | Albert Astals Cid (aacid) → nobody |
| Changed in unity-2d: | |
| status: | In Progress → Invalid |
| Changed in unity-2d (Ubuntu): | |
| status: | Confirmed → Invalid |
| Changed in unity-2d (Ubuntu Precise): | |
| status: | Confirmed → Invalid |
Hi Tom, a patch would be great, but here are instructions on how to submit a merge proposal (which is the preferred way to submit code to Unity) http://
Thanks and good luck!