> On 2010-12-28 10:26:58, Raphael Kubo da Costa wrote: > > /trunk/KDE/kdenetwork/kopete/plugins/history/historylogger.cpp, line 28 > > <http://svn.reviewboard.kde.org/r/6053/diff/1/?file=42267#file42267line28> > > > > Minor nitpick: maybe you should follow the other #includes' style and > > include module/header?
http://doc.qt.nokia.com/4.7/qtextdocument.html I see that it is part of QtGui. So I change it to #include <QtGui/QTextDocument> - Pali ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6053/#review9437 ----------------------------------------------------------- On 2010-12-05 16:23:03, Pali Rohár wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6053/ > ----------------------------------------------------------- > > (Updated 2010-12-05 16:23:03) > > > Review request for Kopete. > > > Summary > ------- > > This patch add support for storing rich html messages in history. Before, all > messages was first converted to plaintext and then saved in history plugin. > Now html messages are saved in html format and are not converted to plaintext. > > > Diffs > ----- > > /trunk/KDE/kdenetwork/kopete/plugins/history/historylogger.cpp 1203573 > > Diff: http://svn.reviewboard.kde.org/r/6053/diff > > > Testing > ------- > > Old history is loaded and displayed correctly. > Messages in plaintext with user writed html tags (If user write plaintext > message which contains < > & ... in chatwindow) are displayed correctly. > Messages in html with < & ... are displayed correctly too. > > > Thanks, > > Pali > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
