It appears that the calculator key shortcut issue isn't an issue with kde-base/kdelibs-4.7.*,4.8.*,4.9* as I originally thought in bug 406049. An upstream developer for KDE mentioned that it's likely a problem with qt-gui. So I created a patch to test the theory, recompiled qt-gui-4.8.2 with the patch, removed the calculator patch from kdelibs and recompiled that. As a result, the bug is also fixed this way but in a better way since when you register the shortcut like in kmenuedit for example, the shortcut shows up as "Calculator" instead of Launch (1). So I believe it would be better if you patch qt-gui instead of kdelibs since whatever depends on qt-gui that might use this code would also be affected. I will attach my patch for qt-gui as well. Reproducible: Always
Created attachment 320800 [details, diff] qt-gui-4.8.2-calculator-key.patch
Created attachment 320806 [details, diff] qt-gui-4.8.2-calculator-key.patch My original patch didn't account for keyboards that register the calculator key as 0x1008FF54. This patch would work for both types of keyboards that register the calculator key as 0x1008FF1D or 0x1008FF54.
I just created a ticket under the upstream qt tracker as well: https://bugreports.qt-project.org/browse/QTBUG-26812 So maybe the guys at KDE (https://bugs.kde.org/show_bug.cgi?id=257900) can push it to QT or the QT devs can handle it.
The only way to submit patches to Qt upstream is through gerrit (https://codereview.qt-project.org/)
(In reply to comment #4) > The only way to submit patches to Qt upstream is through gerrit > (https://codereview.qt-project.org/) Once you've done that, please add a link to the gerrit merge request here, so that we can follow how it evolves.
Closing UPSTREAM per my previous comments.
