----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1094/#review1722 -----------------------------------------------------------
if this is needed (which it may be in this case) then it should not be done using a constructor imo but rather a setNativeWidget method. that a combobox is created and then discarded in the process is a non-issue (it's not a hot path and the execution time of that code is quite negligable). - Aaron On 2009-07-21 15:55:33, Michal Dutkiewicz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/1094/ > ----------------------------------------------------------- > > (Updated 2009-07-21 15:55:33) > > > Review request for Plasma and Marco Martin. > > > Summary > ------- > > This patch adds possibility to use custom combo box widgets (subclassed from > KComboBox) instead of plain KComboBox. > It's intended for 4.3 (better later than never..., didn't know that there is > still no possibility to do something like this and no possibility to create > custom Plasma themed widget). > It adds new constructor to set widget without creating new like in first. > It will be used by Run Command applet (KHistoryComboBox, waiting for theming > possibility for very long time) and probably by others in the future. > If it will be accepted I'll probably prepare similar patches to some other > widgets (most common) for consistency and bigger flexibility that comes > without costs (as far as I know). > > > Diffs > ----- > > /trunk/KDE/kdelibs/plasma/widgets/combobox.h 1000582 > /trunk/KDE/kdelibs/plasma/widgets/combobox.cpp 1000582 > > Diff: http://reviewboard.kde.org/r/1094/diff > > > Testing > ------- > > Tested by my friend, compiles and works. > > > Screenshots > ----------- > > Plasma themed KHistoryComboBox in Run Command applet > http://reviewboard.kde.org/r/1094/s/151/ > > > Thanks, > > Michal > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel