----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104149/#review11318 -----------------------------------------------------------
This review has been submitted with commit 11c7c9adcce7f476b32128f104e2935c6e19ac9d by Daniele E. Domenichelli to branch KDE/4.8. - Commit Hook On March 4, 2012, 2:58 a.m., Daniele Elmo Domenichelli wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104149/ > ----------------------------------------------------------- > > (Updated March 4, 2012, 2:58 a.m.) > > > Review request for kdelibs. > > > Description > ------- > > KWidgetItemDelegate does not update when the itemView gets or loses the focus. > This may cause problems with the text colors of the selected indexes. > > I'm not sure if this is the best way to do it, since in all the other places > this is done asyncronously using "QTimer::singleShot(0, this, > SLOT(initializeModel()))", but this initializes the whole model, and I think > it is not needed here. The alternatives are to update the whole model, or to > add another slot to initialize only selected items. > > > Diffs > ----- > > kdeui/itemviews/kwidgetitemdelegate.cpp 828e498 > > Diff: http://git.reviewboard.kde.org/r/104149/diff/ > > > Testing > ------- > > > Thanks, > > Daniele Elmo Domenichelli > >
