This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6005/

Ship it!

Have you tried removing the space again after you've changed the font? That way it would be transparent I think.

Otherwise, patch is OK from my POV. When you've filed a Qt bug, please put a reference URL to it in the code, as it's a bit "unusual" and easier to keep track that way.

trunk/KDE/kdeplasma-addons/applets/notes/notes.cpp (Diff revision 1)
		void NotesTextEdit::addSpaceToEmptyNotes()
		85	if (toPlainText() == ""){

use toPlainText().isEmpty() here

- Sebastian

On November 28th, 2010, 8:23 p.m., Davide Bettio wrote:

Review request for Plasma and Stephen Kelly.

By Davide Bettio.

Updated 2010-11-28 20:23:45

Description

Text format of the note can't be changed while the note is empty, so I add a space and I change the text format.
This work around is rather stupid but I don't have any nicer idea.
Anyway this might be a bug in KRichTextEdit so I add to this review request also steveire.

Diffs

trunk/KDE/kdeplasma-addons/applets/notes/notes.h (1201054)
trunk/KDE/kdeplasma-addons/applets/notes/notes.cpp (1201054)

View Diff