dfaure added a comment.
Ah right, my comment was wrong, it assumed that `text` was the current text of `tb`, but it's not (necessarily). So your patch is ok, it's just that I'd rather remove all this code with my other patch instead ;) But until it's approved, let's have yours in... REPOSITORY R263 KXmlGui REVISION DETAIL https://phabricator.kde.org/D7954 To: mardelle, #frameworks, ilic, dfaure