----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110825/#review33770 -----------------------------------------------------------
Ship it! Move looks OK. Found some stuff when looking at the header... tier1/kwidgetsaddons/src/keditlistwidget.h <http://git.reviewboard.kde.org/r/110825/#comment24893> Should be removed from the public header. tier1/kwidgetsaddons/src/keditlistwidget.h <http://git.reviewboard.kde.org/r/110825/#comment24894> Oh man, these docs are old :) - David Faure On June 4, 2013, 2:54 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110825/ > ----------------------------------------------------------- > > (Updated June 4, 2013, 2:54 p.m.) > > > Review request for KDE Frameworks. > > > Description > ------- > > Moved KEditListWidget to KWidgetAddons. > > I changed from KLineEdit to QLineEdit, which I think it's the one that we're > supposed to have. Everything worked, but QLineEdit doesn't have > setTrapReturnKey. I added code in the event filter to trap the lineedit's > return key. > > > Diffs > ----- > > kdeui/CMakeLists.txt 6e7b7ee > kdeui/tests/CMakeLists.txt 6fca0bb > kdeui/tests/keditlistwidgettest.cpp ad1a97d > kdeui/widgets/keditlistwidget.h dfc16da > kdeui/widgets/keditlistwidget.cpp 075bac3 > kio/kfile/kurlrequester.h 0b07373 > tier1/kwidgetsaddons/src/CMakeLists.txt 9802467 > tier1/kwidgetsaddons/src/keditlistwidget.h PRE-CREATION > tier1/kwidgetsaddons/src/keditlistwidget.cpp PRE-CREATION > tier1/kwidgetsaddons/tests/CMakeLists.txt bd00be8 > tier1/kwidgetsaddons/tests/keditlistwidgettest.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/110825/diff/ > > > Testing > ------- > > everything builds, the test seems to work > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
