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

List:       koffice-devel
Subject:    Re: Thesaurus KDataTool
From:       Daniel Naber <daniel.naber () t-online ! de>
Date:       2001-11-08 0:43:59
[Download RAW message or body]

On Wednesday 07 November 2001 23:31, David Faure wrote:

> Right, that's the problem I had in mind when saying that :}
>
> Actually, passing an empty string to KAction should do the right thing
> IMHO.
>
> Does this help ?

No, the string is "unknown". So it works with this. Can I apply?

BTW, using *only* the keyboard will not use the "current" word (the word 
the cursor is over) but an old word which was previously checked. In other 
words, using the keyboard to access the thesaurus currently only works 
correctly when the word is selected.

regards
 Daniel

-- 
http://www.danielnaber.de

["no_icon.diff" (text/x-diff)]

Index: kaction.cpp
===================================================================
RCS file: /home/kde/kdelibs/kdeui/kaction.cpp,v
retrieving revision 1.196
diff -u -r1.196 kaction.cpp
--- kaction.cpp	2001/10/29 18:52:51	1.196
+++ kaction.cpp	2001/11/08 00:39:48
@@ -199,8 +199,8 @@
     setAccel( accel );
 
     d->m_iconName=pix;
-    d->m_bIconSetNotYetLoaded=true;
-    d->m_bIconSet=true;
+    d->m_bIconSetNotYetLoaded=!(pix == "unknown");
+    d->m_bIconSet=!(pix == "unknown");
 }
 
 KAction::KAction( const QString& text, const QIconSet& pix, int accel,
@@ -241,9 +241,9 @@
     setText( text );
 
     d->m_iconName=pix;
-    d->m_bIconSetNotYetLoaded=true;
-    d->m_bIconSet=true;
-
+    d->m_bIconSetNotYetLoaded=!(pix == "unknown");
+    d->m_bIconSet=!(pix == "unknown");
+    
     if ( receiver )
       connect( this, SIGNAL( activated() ), receiver, slot );
 }

_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/koffice-devel


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