[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: kmenuedit disallows Meta key?
From:       Lubos Lunak <l.lunak () sh ! cvut ! cz>
Date:       2001-06-28 7:27:54
[Download RAW message or body]

Dne ?t 28. ?erven 2001 02:36 Malte Starostik napsal(a):
> Hi,
>
> I just tried to assign a shortcut with Meta to an app in the K-Menu and
> kmenuedit claimed the Meta key was not allowed in this context. Is there
> any reason for this? According to the khotkeys sources, false is passed for
> allowMeta to the KKeyChooser dialog in this case. Is this a typo or by
> design?
>

 It is not a typo and it's not by design, it's by the fact that the default 
value is false. Is this the change that should make it work ?

--- kcmkhotkeys.cpp.bak Mon Sep 25 21:51:22 2000
+++ kcmkhotkeys.cpp     Thu Jun 28 09:22:14 2001
@@ -213,7 +213,7 @@ desktop_shortcut_dialog::desktop_shortcu
     line->setText( item_P->run );
 //    line->setMinimumWidth( 500 );
     line->setReadOnly( true );
-    keychooser = new KKeyChooser( &map, page, true );
+    keychooser = new KKeyChooser( &map, page, true, false, true );
     connect( keychooser, SIGNAL( keyChange()), this, SLOT( key_changed()));
     QBoxLayout* main_layout = new QVBoxLayout( page, KDialog::marginHint(),
         KDialog::spacingHint());

 I cannot check it ... heck, I cannot even check if it compiles, I have my 
computer at home for the holidays. Please commit it if it's the correct fix.

 Lubos Lunak
--
 l.lunak@email.cz     l.lunak@kde.org
 http://dforce.sh.cvut.cz/~seli

[prev in list] [next in list] [prev in thread] [next in thread]