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