shortcut-conflict workaround for bug #165938

Sebastian Sauer mail at dipe.org
Tue Dec 16 03:22:53 CET 2008


Hi *,

http://bugs.kde.org/show_bug.cgi?id=165938 is about some shortcuts like CTRL+A 
(select all) and CTRL+P (paste) are not usuable in qlineedit cause they are 
already registered for the "Add Widgets" dialog and the "Select previous 
widget" action.

As far as I understood, that will be probably fixed in Qt 4.5 but since Qt 4.5 
is not out yet and since KDE 4.2 users will go with Qt 4.3, I would suggest 
following workaround for KDE 4.2 to at least have selectall+paste working 
fine in KDE4.2 at Qt4.3 ...

Would that be ok to commit?

Index: containment.cpp
===================================================================
--- containment.cpp     (revision 897266)
+++ containment.cpp     (working copy)
@@ -148,7 +148,7 @@
     appletBrowserAction->setEnabled(unlocked);
     connect(appletBrowserAction, SIGNAL(triggered()), this, 
SLOT(triggerShowAddWidgets()));
     appletBrowserAction->setShortcutContext(Qt::WidgetShortcut);
-    appletBrowserAction->setShortcut(QKeySequence("ctrl+a"));
+    //appletBrowserAction->setShortcut(QKeySequence("ctrl+a")); //FIXME in 
4.3, see bug #165938
     d->actions().addAction("add widgets", appletBrowserAction);

     QAction *configureActivityAction = new QAction(i18n("Appearance 
Settings"), this);
@@ -169,7 +169,7 @@
     //no icon
     connect(action, SIGNAL(triggered()), this, SLOT(focusPreviousApplet()));
     action->setShortcutContext(Qt::WidgetShortcut);
-    action->setShortcut(QKeySequence("ctrl+p"));
+    //action->setShortcut(QKeySequence("ctrl+p")); //FIXME in 4.3, see bug 
#165938
     d->actions().addAction("previous applet", action);

     if (immutability() != SystemImmutable) {


More information about the Plasma-devel mailing list