Am 18.06.11 16:44 schrieb(en) Albrecht Dreß:
Am 18.06.11 16:24 schrieb(en) Oliver Eichler:
Not really harmful. When the automated part of the diary is created the 
undo/redo stack is filled. If you do an undo the tables and stuff start to 
vanish. That is not what the user would expect. But you can a redo again. Thus 
it's more a cosmetic issue.

Couldn't we achieve the same effect by using the sequence
        setUndoRedoEnabled(false);
        // fill automated part
        setUndoRedoEnabled(true);
which seems to be compatible with 4.6?

I'll test it and submit a patch if it works...

See below - afaict, the application behaves identically (tested w/ Ubuntu, 
where I have 4.7).

Opinions?

Best, Albrecht.
Index: src/CDiaryEdit.cpp
===================================================================
--- src/CDiaryEdit.cpp	(Revision 2839)
+++ src/CDiaryEdit.cpp	(Arbeitskopie)
@@ -512,6 +512,8 @@ void CDiaryEdit::draw(QTextDocument& doc)
 
     if(pointSize == 0) return;
 
+    doc.setUndoRedoEnabled(false);
+
     QFont f = textEdit->font();
     f.setPointSize(pointSize);
     textEdit->setFont(f);
@@ -689,7 +691,7 @@ void CDiaryEdit::draw(QTextDocument& doc)
 
         }
     }
-    doc.clearUndoRedoStacks();
+    doc.setUndoRedoEnabled(true);
 }
 
 //static QString toPlainText(const QTextTableCell& cell)

Attachment: pgpVwnrJHETLJ.pgp
Description: PGP signature

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
Qlandkartegt-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qlandkartegt-users

Reply via email to