unity-api-bugs team mailing list archive

[Dmitry Shachnev <mitya57@xxxxxxxxx>]

** Description changed:

- In some apps built using QT4 & 5, menu shortcuts are greyed out and
- inoperant. Only alt and FKeys-based shortcuts work. Others, notably
- ctrl+c for copying, do not.
+ Impact
+ ======
  
- This is quite serious ; mail me for more details.
+ This affects all Qt applications on Unity and other desktops which use
+ global menu, either via appmenu-qt5 or via native implementation on
+ Yakkety.
+ 
+ Any shortcut is not working if the corresponding action is attached only
+ to the menubar (i.e. and not also to the toolbar). It can be reproduced
+ with almost any application by removing the toolbar (right-clicking on
+ it and deselecting it).
+ 
+ Test Case
+ =========
+ 
+ * Open Qt Assistant;
+ * Press Ctrl+T (this action is not on the toolbar).
+ 
+ Expected: a new tab should be opened. Current result: nothing happens.
+ 
+ Proposed Fix
+ ============
+ 
+ The proposed fix is a backport of the upstream fix at
+ https://code.qt.io/cgit/qt/qtbase.git/commit/?id=287f548d4c7cc594.
+ 
+ Actually just the qshortcut.cpp part would be sufficient, but the other
+ parts make sure it plays fine together with https://cgit.kde.org/plasma-
+ integration.git/commit/?id=aef74e97e2ed462a.
+ 
+ Regression Potential
+ ====================
+ 
+ The fix is in Zesty for 1½ months now (in upstream even longer), and so
+ far nobody complained. People have verified that the fix works (see
+ comment #63).

** Description changed:

  Impact
  ======
  
  This affects all Qt applications on Unity and other desktops which use
  global menu, either via appmenu-qt5 or via native implementation on
  Yakkety.
  
  Any shortcut is not working if the corresponding action is attached only
  to the menubar (i.e. and not also to the toolbar). It can be reproduced
  with almost any application by removing the toolbar (right-clicking on
  it and deselecting it).
  
  Test Case
  =========
  
  * Open Qt Assistant;
  * Press Ctrl+T (this action is not on the toolbar).
  
  Expected: a new tab should be opened. Current result: nothing happens.
  
+ In Yakkety the fix should work both with and without appmenu-qt5.
+ 
  Proposed Fix
  ============
  
  The proposed fix is a backport of the upstream fix at
  https://code.qt.io/cgit/qt/qtbase.git/commit/?id=287f548d4c7cc594.
  
  Actually just the qshortcut.cpp part would be sufficient, but the other
  parts make sure it plays fine together with https://cgit.kde.org/plasma-
  integration.git/commit/?id=aef74e97e2ed462a.
  
  Regression Potential
  ====================
  
  The fix is in Zesty for 1½ months now (in upstream even longer), and so
  far nobody complained. People have verified that the fix works (see
  comment #63).

-- 
You received this bug notification because you are a member of Unity API
bugs, which is subscribed to sni-qt.
https://bugs.launchpad.net/bugs/1380702

Title:
  No keyboards shortcuts in QT apps

Status in appmenu-qt5:
  In Progress
Status in Canonical System Image:
  In Progress
Status in sni-qt:
  New
Status in qtbase-opensource-src package in Ubuntu:
  Fix Released
Status in qtbase-opensource-src source package in Xenial:
  Confirmed
Status in qtbase-opensource-src source package in Yakkety:
  Confirmed

Bug description:
  Impact
  ======

  This affects all Qt applications on Unity and other desktops which use
  global menu, either via appmenu-qt5 or via native implementation on
  Yakkety.

  Any shortcut is not working if the corresponding action is attached
  only to the menubar (i.e. and not also to the toolbar). It can be
  reproduced with almost any application by removing the toolbar (right-
  clicking on it and deselecting it).

  Test Case
  =========

  * Open Qt Assistant;
  * Press Ctrl+T (this action is not on the toolbar).

  Expected: a new tab should be opened. Current result: nothing happens.

  In Yakkety the fix should work both with and without appmenu-qt5.

  Proposed Fix
  ============

  The proposed fix is a backport of the upstream fix at
  https://code.qt.io/cgit/qt/qtbase.git/commit/?id=287f548d4c7cc594.

  Actually just the qshortcut.cpp part would be sufficient, but the
  other parts make sure it plays fine together with https://cgit.kde.org
  /plasma-integration.git/commit/?id=aef74e97e2ed462a.

  Regression Potential
  ====================

  The fix is in Zesty for 1½ months now (in upstream even longer), and
  so far nobody complained. People have verified that the fix works (see
  comment #63).

To manage notifications about this bug go to:
https://bugs.launchpad.net/appmenu-qt5/+bug/1380702/+subscriptions