Dash, multi-monitor - Dash jumps from one monitor to another in a multi-monitor environment
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ayatana Design |
Fix Released
|
Critical
|
John Lea | ||
Unity |
Fix Released
|
Critical
|
Brandon Schaefer | ||
unity (Ubuntu) |
Fix Released
|
Critical
|
Brandon Schaefer |
Bug Description
Dash jumps from one monitor to another in a multi-monitor environment. There are two ways to reproduce this issue:
-------
First way to reproduce:
1. Open GEdit on the right monitor
2. Open the Dash on the left monitor
3. Leaving the Dash open, click the save button in GEdit
See attached video "dash_jumps.ogv".
-------
Second way to reproduce:
1. Open the Dash on the left monitor
2. Type one character
3. Move the pointer to the right monitor
4. Type another character
Result: The Dash jumps from one monitor to another, this is incorrect.
See attached video "dash_jumps2.ogv"
-------
Current incorrect behaviour: The Dash jumps from one monitor to another, and on the second monitor shows an empty Launcher!
What should happen:
- The Dash should close when the user clicks anywhere that is not inside the Dash on *any* monitor. The click action should trigger whatever was clicked on, e.g. in the case above, clicking on the GEdit "save" button should save the text file and close the Dash at the same time.
- Once the Dash is open, it should *never* jump to another monitor. The Dash should always remain on the monitor it was opened on.
- Fixing this bug will also fix the issue with ghost images of windows that are no longer behind the dash appearing behind the Dash in multi-monitor environments (watch the background behind the Dash in screencast dash_jumps2.ogv to see this issue). Fixing this bug will remove the possibly of the user entering the state where these ghost images can occur.
Related branches
- jenkins (community): Needs Fixing (continuous-integration)
- Tim Penhey (community): Approve
-
Diff: 365 lines (+88/-32)9 files modifieddash/DashController.cpp (+7/-2)
dash/DashView.cpp (+6/-6)
hud/HudController.cpp (+14/-8)
hud/HudController.h (+1/-1)
hud/HudView.cpp (+3/-0)
tests/autopilot/unity/emulators/dash.py (+7/-2)
tests/autopilot/unity/emulators/hud.py (+5/-0)
tests/autopilot/unity/tests/test_dash.py (+27/-11)
tests/autopilot/unity/tests/test_hud.py (+18/-2)
tags: | added: udp |
Changed in unity: | |
milestone: | none → backlog |
Changed in ayatana-design: | |
assignee: | nobody → John Lea (johnlea) |
importance: | Undecided → Critical |
status: | New → Triaged |
Changed in unity: | |
status: | New → Triaged |
Changed in unity (Ubuntu): | |
status: | New → Triaged |
tags: | added: multimonitor |
Changed in ayatana-design: | |
status: | Triaged → Fix Committed |
Changed in unity: | |
importance: | Undecided → High |
description: | updated |
description: | updated |
Changed in unity (Ubuntu): | |
importance: | Undecided → High |
tags: | added: dashm |
Changed in unity: | |
importance: | High → Critical |
Changed in unity (Ubuntu): | |
importance: | High → Critical |
Changed in unity: | |
assignee: | nobody → Brandon Schaefer (brandontschaefer) |
Changed in unity (Ubuntu): | |
assignee: | nobody → Brandon Schaefer (brandontschaefer) |
Changed in unity: | |
status: | Triaged → Confirmed |
status: | Confirmed → In Progress |
Changed in unity (Ubuntu): | |
status: | Triaged → In Progress |
Changed in unity: | |
milestone: | backlog → 6.6 |
Changed in unity: | |
status: | In Progress → Fix Committed |
Changed in unity (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in unity: | |
milestone: | 6.6 → 7.0 |
no longer affects: | unity/6.0 |
Changed in unity: | |
milestone: | 7.0 → 6.6 |
Changed in unity: | |
status: | Fix Committed → Fix Released |
Changed in ayatana-design: | |
status: | Fix Committed → Fix Released |
tags: |
added: reviewedbydesignq removed: udp |
This bug was fixed in the package unity - 6.6.0-0ubuntu1
---------------
unity (6.6.0-0ubuntu1) quantal-proposed; urgency=low
* New upstream release. rface (LP: #876017) rateColor to a void division by zero. (LP: #975350)
unityshell. cpp static constants. COMPIZ_ GL ifdefs and remove
- Fixes non-escaped character sequences in dash previews (LP: #1039020)
- Updated background layer for preview cover-art and details panels
to be 10% low-light
- Expand a PlacesGroup if it is the only category that contains results
(LP: #950710)
- Update unity autopilot tests to match autopilot API
- Updated the convert files to fix some typos in the key names
- Add gmodule dependency
- Activate proper result if the categories aren't displayed in-order
(LP: #1040101)
- Refactor device launcher icons (LP: #713423)
- LauncherController: make the controller enable the launcher struts,
based on hide-mode option (LP: #1044005)
- Launcher: make always possible to drag an icon to the bottom or top
of its sub list (LP: #1043968)
- Don't desat bfb/hud icon in DNDReset (LP: #1043963)
- Progressively adjust the speed of the minimize animation. First
time it is used is slower, then speeds up the more it is used.
(LP: #1017510)
- Implement new ordering of categories for home lens. (LP: #1043915)
- UnityWindow now implements ScaleWindowInte
- Launcher: restore an icon position after that the dragging has been
cancelled (LP: #955561)
- LauncherDragWindow: cancel drag on window mapped/unmapped
(LP: #1044723)
- Now there is a check of an override color in RefreshColor (which is
called when a PropertyNotify event happens). Also added a check in
FullySatu
- Queue redraw after cover-art texture is updated from a url/file source.
(LP: #1043947)
- Fixed ability to delete glib::Source wrapper during its callback
(LP: #1044823)
- Close preview when dash is hidden. (LP: #1045298)
- LauncherModel: rewrite the Reordering functions to keep the icon
priority deltas (LP: #761155)
- Make sure we can pass extra hints when activating preview actions.
(LP: #1046352)
- UnityWindow: scale window code improved (LP: #1033935)
- The mouse will now cause the HUD buttons to change selection
(LP: #1042692)
- "Alt+Space" shortcut to reveal the window menu is not hardcoded, but a
Compiz key option. " (Hold)" should also be translated. Made all
Compiz plug-in names and all Compiz plug-in option names in
- remove unity --reset, it's not anymore really needed now that we are
in stable days of unity and we moved to gsettings
- Removed the variables 'oldPrev' and 'oldNext' which got assigned the
value NULL, but then were never used
- Fixed the size of the previews to 770x380 pixels. (LP: #1045243)
- UnityWindow: use smart pointers, use static close_icon (with dynamic
state) and PanelStyle context (LP: #1033935) (LP: #1045127)
(LP: #1046124) (LP: #1046126)
- Remove everything in the #ifndef USE_MODERN_
the ifdefs alltogether. unity now requires compiz...