[regression] Shortcut overlay does not hide when modifier key is pressed

Bug #1075207 reported by Łukasz Zemczak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ayatana Design
Fix Committed
High
John Lea
Compiz
Fix Released
High
Brandon Schaefer
0.9.9
Fix Released
High
Brandon Schaefer
Unity
Fix Released
High
Unassigned
compiz (Ubuntu)
Fix Released
High
Brandon Schaefer
unity (Ubuntu)
Fix Released
High
Brandon Schaefer

Bug Description

On trunk, the autopilot test case unity.tests.test_shortcut_hint.ShortcutHintInteractionsTests.test_shortcut_hint_hide_pressing_modifiers fails as a regression.

[Test Case]
1. Press and hold Super to show the Shortcut overlay screen
2. Press any modifier key
   -> Make sure the overlay badges on the launcher icons disappear

----------------------------------
Desired solution:

- When the SUPER key is pressed, all keys with the exception of the "P" key should be grabbed by Unity.

- This should allow the "Shortcut overlay does not hide when modifier key is pressed" while enabling the laptop SUPER+P presentation mode to still work.

Related branches

Revision history for this message
Łukasz Zemczak (sil2100) wrote :
Changed in unity:
milestone: none → 7.0.0
Andrea Azzarone (azzar1)
Changed in unity:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Testing on quantal the Keyboard Shortcuts overlay does disappear correctly. It's only the overlay badges on the launcher that don't disappear. Is that what you mean?

Changed in unity:
status: Triaged → Incomplete
Revision history for this message
John Lea (johnlea) wrote :

@vanvugt; the overlay badges should also disappear so this is a regression.

Changed in ayatana-design:
status: New → Fix Committed
assignee: nobody → John Lea (johnlea)
importance: Undecided → High
description: updated
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

@Daniel
For me both the overlay and the badges don't disappear when I tap Alt, Ctrl or Shift while holding Super. Is it differently for you? I'm currently using unity staging, but I had the same thing when using quantal.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Hmm, OK. In that case I can't reproduce the whole bug :/

Changed in unity:
status: Incomplete → New
Revision history for this message
Stephen M. Webb (bregma) wrote :

I can confirm this occurs on Raring, but it does not occur on Quantal.

Changed in unity:
status: New → Triaged
Changed in unity (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
John Lea (johnlea)
summary: - Shortcut overlay does not hide when modifier key is pressed
+ [regression] Shortcut overlay does not hide when modifier key is pressed
Changed in unity:
importance: Medium → High
Changed in unity (Ubuntu):
importance: Medium → High
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

This is caused by fixing this bug:
https://bugs.launchpad.net/ubuntu/quantal/+source/unity/+bug/950160

So the only way we can fix this is by reverting that other fix... The problem being we no longer grab all the keys so when a modifier is pressed compiz doesn't send the key terminate....

Revision history for this message
John Lea (johnlea) wrote :

@brandontschaefer; +1 from me for reverting the fix to bug #950160 to solve this issue. Note we only want to grab all other keys when SUPER is pressed, we should not grab all other keys when any other modifier is pressed.

Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

@John; Well the problem is, if we grab all the keys when super is pressed Super+P no longer works for going into presentation mode. Which the Super+P is not changeable...so if we grab all the keys when Super is pressed, Super+P no longer works which is one of the main problems with that other bug....so we are kind of in a catch 22 situation.

Revision history for this message
John Lea (johnlea) wrote :

@brandontschaefer; can we grab all keys other than SUPER+P and create an exception for this one shortcut?

Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

We would have to hard code that in... and I think the only way that would get accepted is by adding this to the ubuntu patches...but Ill have to talk with Daniel or Sam about it.

Revision history for this message
John Lea (johnlea) wrote :

@brandontschaefer; sounds good, fingers crossed you can find a way to do it. I've updated the bug description.

description: updated
Changed in unity:
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in unity (Ubuntu):
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in unity:
status: Triaged → In Progress
Changed in unity (Ubuntu):
status: Triaged → In Progress
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

@John; After talking with Daniel Ill implement a fix where we only ungrab Super+P leaving OEM (Dell) happy, while grabbing everything else which will fix this problem. So it should be a compiz only fix.

Changed in compiz:
status: New → Triaged
Changed in compiz (Ubuntu):
status: New → In Progress
Changed in compiz:
status: Triaged → In Progress
importance: Undecided → High
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in compiz (Ubuntu):
assignee: nobody → Brandon Schaefer (brandontschaefer)
Changed in compiz:
milestone: none → 0.9.9.2
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:compiz/raring at revision 3633, scheduled for release in compiz, milestone Unknown

Changed in compiz:
status: In Progress → Fix Committed
Changed in unity:
status: In Progress → Fix Committed
Changed in unity (Ubuntu):
status: In Progress → Fix Committed
Changed in compiz (Ubuntu):
status: In Progress → Fix Committed
importance: Undecided → High
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:compiz at revision None, scheduled for release in compiz, milestone 0.9.9.2

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.9~daily13.03.08-0ubuntu1

---------------
compiz (1:0.9.9~daily13.03.08-0ubuntu1) raring; urgency=low

  [ Brandon Schaefer ]
  * [regression] Shortcut overlay does not hide when modifier key is
    pressed (LP: #1075207)
  * Unity blocks other programs from binding globally to Super+* (* =
    any key) (LP: #950160)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 3633
 -- Automatic PS uploader <email address hidden> Fri, 08 Mar 2013 10:34:21 +0000

Changed in compiz (Ubuntu):
status: Fix Committed → Fix Released
Changed in compiz:
milestone: 0.9.9.2 → 0.9.10.0
Revision history for this message
Stephen M. Webb (bregma) wrote :

Fix Released in Unity Unity 7.0.0 "R series".

Changed in unity:
status: Fix Committed → Fix Released
Stephen M. Webb (bregma)
Changed in compiz:
status: Fix Committed → Fix Released
Changed in unity (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Stephen M. Webb (bregma) wrote :

Closing Compiz 0.9.9 task (0.9.9 series is obsolete)

tags: added: rls-w-incoming
Changed in unity:
assignee: Brandon Schaefer (brandontschaefer) → Daniyaal Rasheed (radonapps)
Changed in unity:
assignee: Daniyaal Rasheed (radonapps) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.