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

List:       kde-devel
Subject:    Fix for bug #156122
From:       Shai Berger <shai () platonix ! com>
Date:       2008-10-24 14:28:46
Message-ID: 200810241628.46437.shai () platonix ! com
[Download RAW message or body]

Hi list,

The attached patch fixes bug #156122, which severely limits kolourpaint for 
non-iso-latin-1 users on UTF locales. Can I commit? Or is there some other 
place to ask?

Thanks,
	Shai.

["kolourpaint.patch" (text/x-diff)]

Index: kolourpaint/tools/selection/text/kpToolText_KeyboardEvents.cpp
===================================================================
--- kolourpaint/tools/selection/text/kpToolText_KeyboardEvents.cpp	(revision 874618)
+++ kolourpaint/tools/selection/text/kpToolText_KeyboardEvents.cpp	(working copy)
@@ -89,7 +89,7 @@
     // TODO: don't like this check
     else if ((ke->modifiers () &
                 (Qt::ControlModifier | Qt::AltModifier | Qt::MetaModifier)) == 0 &&
-            ke->key () < 0x100 /*QChar (ke->key ()).isPrint () - unfortunately F1 is printable too...*/)
+             !( ke->text ().isEmpty ()))
     {
     #if DEBUG_KP_TOOL_TEXT
         kDebug () << "\tis text - grab";


>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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