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

List:       koffice-devel
Subject:    PATCH: kotext2: backspace is better when you can _see_ it work
From:       "Benjamin K. Stuhl" <benjamin.stuhl () colorado ! edu>
Date:       2006-08-19 16:56:57
Message-ID: 200608191057.14386.benjamin.stuhl () colorado ! edu
[Download RAW message or body]

[Attachment #2 (multipart/signed)]

[Attachment #4 (multipart/mixed)]


Right now, backspace and delete don't actually update the screen to show the 
changed text; the attached patch fixes that by giving them the same repaint() 
treatment that we do for text insertion.While we're at it, delete and 
backspace should also hide the cursor, since they too indicate an intent to 
sit down and edit the text.

-- BKS

["kotext2-fix-deleted.patch" (text/x-diff)]

Index: koffice/libs/kotext/KoTextTool.cpp
===================================================================
--- koffice/libs/kotext/KoTextTool.cpp	(revision 574643)
+++ koffice/libs/kotext/KoTextTool.cpp	(working copy)
@@ -155,12 +155,16 @@
     QTextCursor::MoveOperation moveOperation = QTextCursor::NoMove;
     switch(event->key()) { // map input to moveOperation
         case Qt::Key_Backspace:
+            repaint();
+            useCursor(Qt::BlankCursor);
             m_caret.deletePreviousChar();
             break;
         case Qt::Key_Tab:
             kDebug(32500) << "Tab key pressed";
             break;
         case Qt::Key_Delete:
+            repaint();
+            useCursor(Qt::BlankCursor);
             m_caret.deleteChar();
             break;
         case Qt::Key_Left:

[Attachment #8 (application/pgp-signature)]

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


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