[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-07-14 Thread Mykola Krachkovsky
https://bugs.kde.org/show_bug.cgi?id=375053

Mykola Krachkovsky  changed:

   What|Removed |Added

 CC||w01dn...@gmail.com

--- Comment #13 from Mykola Krachkovsky  ---
Bug still present.
openSUSE Tumbleweed x86_64, Qt 5.9.1, KDE Plasma 5 with global menu on.

As workaround I'm using `KDE_NO_GLOBAL_MENU=1 qupzilla` to start it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-07-25 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

David Rosca  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
  Latest Commit|https://commits.kde.org/pla |https://commits.kde.org/pla
   |sma-workspace/c24ae6602b0d0 |sma-workspace/dba0f80288b41
   |1cb629e17c4789cecf8c15bd048 |e385f6a358f325b6b05d8ae11c4
 Resolution|--- |FIXED
   Version Fixed In||5.11.0

--- Comment #14 from David Rosca  ---
Git commit dba0f80288b41e385f6a358f325b6b05d8ae11c4 by David Rosca.
Committed on 25/07/2017 at 15:25.
Pushed by drosca into branch 'master'.

AppMenu: Don't call AboutToShow on LayoutUpdated

This workaround breaks applications that rely on aboutToShow/aboutToHide
signals, because it sends aboutToShow to all menus on startup without actually
the menu being opened.
FIXED-IN: 5.11.0

Differential Revision: https://phabricator.kde.org/D5705

M  +0-43   appmenu/menuimporter.cpp
M  +0-3appmenu/menuimporter.h

https://commits.kde.org/plasma-workspace/dba0f80288b41e385f6a358f325b6b05d8ae11c4

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-12-25 Thread acidrums4
https://bugs.kde.org/show_bug.cgi?id=375053

acidrums4  changed:

   What|Removed |Added

 CC||acidrums4+st...@gmail.com

--- Comment #15 from acidrums4  ---
Though it's been marked resolved, I can still reproduce this, despite using
plasma-workspace-5.11.4, Qupzilla 2.2.3 and Qt 5.9.3 (on a Gentoo amd64 box).

The `KDE_NO_GLOBAL_MENU=1 qupzilla` did worked for me, though.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-12-25 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=375053

Antonio Rojas  changed:

   What|Removed |Added

 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #16 from Antonio Rojas  ---
Still valid

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2018-01-19 Thread acidrums4
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #17 from acidrums4  ---
Weirdly enough, this was solved for me with today's update on Qupzilla (2.2.4).
It feels like Global Menu wasn't the only one to blame...

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2018-01-19 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #18 from David Rosca  ---
It is still to blame, but I just changed the behavior in QupZilla that
triggered this bug.
So you should not have this issue anymore, but global menu is still broken - it
incorrectly sends aboutToShow/aboutToHide events (on the Qt side).

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2018-01-29 Thread Alexander Mentyu
https://bugs.kde.org/show_bug.cgi?id=375053

Alexander Mentyu  changed:

   What|Removed |Added

 CC||notux...@gmail.com

--- Comment #19 from Alexander Mentyu  ---
Can't reproduce in:

Plasma: 5.11.5
Apps: 17.12.1
Qt: 5.10.0
Frameworks: 5.42.0
Kernel: 4.14.14-1-MANJARO
QupZilla 2.2.4

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2018-01-29 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

David Rosca  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|REOPENED|RESOLVED

--- Comment #20 from David Rosca  ---
Alright, as far as this bug is concerned, it actually was fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-05-03 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #11 from Antonio Rojas  ---
Seems to be fixed for real with Qt 5.9

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-05-04 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #12 from David Rosca  ---
> Seems to be fixed for real with Qt 5.9

I don't think so. Maybe the behavior of ambiguous shortcuts were changed in Qt
5.9, but the root issue with appmenu is still there.

This fixes this issue: https://phabricator.kde.org/D5705

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-03-16 Thread Vladyslav Tronko
https://bugs.kde.org/show_bug.cgi?id=375053

Vladyslav Tronko  changed:

   What|Removed |Added

 CC||innerm...@gmail.com

--- Comment #7 from Vladyslav Tronko  ---
I confirm this.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-03-25 Thread Michael D
https://bugs.kde.org/show_bug.cgi?id=375053

Michael D  changed:

   What|Removed |Added

 CC||nortex...@gmail.com

--- Comment #8 from Michael D  ---
Does this relate to the fact that global menu breaks shortcuts in a variety of
qt applications, including VLC and Sayonara?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-03-29 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=375053

David Edmundson  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/pla
   ||sma-workspace/c24ae6602b0d0
   ||1cb629e17c4789cecf8c15bd048
 Resolution|--- |FIXED
 Status|REOPENED|RESOLVED

--- Comment #9 from David Edmundson  ---
Git commit c24ae6602b0d01cb629e17c4789cecf8c15bd048 by David Edmundson.
Committed on 29/03/2017 at 15:35.
Pushed by davidedmundson into branch 'Plasma/5.9'.

Connect aboutToHide signal from QMenu to relevant libdbusmenu-qt slot

Summary:
This sends the close event to the client which then sends aboutToHide to
the QMenu.

Code was all there, just not hooked up

Test Plan: Added qDebug to the test app inside libdbusmenuqt

Reviewers: #plasma, markg, broulik

Reviewed By: #plasma, markg, broulik

Subscribers: broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D5192

M  +1-0libdbusmenuqt/dbusmenuimporter.cpp

https://commits.kde.org/plasma-workspace/c24ae6602b0d01cb629e17c4789cecf8c15bd048

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-03-29 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=375053

Antonio Rojas  changed:

   What|Removed |Added

 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #10 from Antonio Rojas  ---
This commit doesn't fix the problem

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-04-07 Thread Luca Giambonini
https://bugs.kde.org/show_bug.cgi?id=375053

Luca Giambonini  changed:

   What|Removed |Added

 CC||luc...@bluewin.ch

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-01-26 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=375053

David Edmundson  changed:

   What|Removed |Added

 CC||k...@davidedmundson.co.uk

--- Comment #1 from David Edmundson  ---
I did try installing qupzilla. For me copy and paste still worked.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-01-28 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #2 from Antonio Rojas  ---
I can still reproduce it with a clean user. 
Qupzilla 2.0.2, Qt 5.8.0, Frameworks 5.30, es_ES.utf8 locale

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-02-13 Thread Antonio Rojas
https://bugs.kde.org/show_bug.cgi?id=375053

Antonio Rojas  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #3 from Antonio Rojas  ---
Fixed with Frameworks 5.31

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-02-22 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

David Rosca  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---
 Ever confirmed|0   |1
 CC||now...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-02-22 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #4 from David Rosca  ---
It's not fixed. The problem is that the QMenu's don't receive aboutToHide
signals when global menu is enabled (at least with global menu in window
titlebars).

I didn't look into global menu code, so this may also be bug in Qt.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-02-23 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #5 from David Edmundson  ---
Why would that break control+C actions?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 375053] enabling global menu breaks copy/paste shortcuts in qupzilla

2017-02-24 Thread David Rosca
https://bugs.kde.org/show_bug.cgi?id=375053

--- Comment #6 from David Rosca  ---
Because it disables those actions in aboutToShow, and when the aboutToShow is
not called, these actions are duplicated and it results in that warning dialog.

Also, why is the dialog shown in Qt-only application?

-- 
You are receiving this mail because:
You are watching all bug changes.