> On July 22, 2015, 9:57 p.m., Pali Rohár wrote: > > plugins/texteffect/texteffectpreferences.cpp, line 160 > > <https://git.reviewboard.kde.org/r/124094/diff/6/?file=384109#file384109line160> > > > > Definition of K3ListBox::insertItem from > > http://doc.qt.io/qt-4.8/q3listbox.html#insertItem-3: > > > > void Q3ListBox::insertItem(const QString & text, int index = -1) > > > > Inserts a new list box text item with the text text into the list at > > position index. If index is negative, text is inserted at the end of the > > list. > > > > Definition of QListWidget::insertItem from > > http://doc.qt.io/qt-4.8/qlistwidget.html#insertItem-2 > > > > void QListWidget::insertItem(int row, const QString & label) > > > > Inserts an item with the text label in the list widget at the position > > given by row. > > > > So for me it looks like your code is incorrect. If not, show me proof.
You're right, however with the current patch, the colors still get added to the end of the list. Strange. - R.Harish ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124094/#review82820 ----------------------------------------------------------- On July 10, 2015, 6:24 p.m., R.Harish Navnit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124094/ > ----------------------------------------------------------- > > (Updated July 10, 2015, 6:24 p.m.) > > > Review request for Kopete, Laurent Montel and Pali Rohár. > > > Repository: kopete > > > Description > ------- > > Replace the K3ListBox classes with QListWidget. > > > Diffs > ----- > > plugins/texteffect/texteffectpreferences.cpp > 35fac60d419cbac5644a8e143be1fc7c640385a9 > plugins/texteffect/texteffectprefs.ui > 1b82f3de4b449373a3dedb39b664c0f645f1d02a > > Diff: https://git.reviewboard.kde.org/r/124094/diff/ > > > Testing > ------- > > Build succeeds. > > > Thanks, > > R.Harish Navnit > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
