----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126996/#review92109 -----------------------------------------------------------
Fix it, then Ship it! One minor question imports/activitymanager/switcherbackend.cpp (line 125) <https://git.reviewboard.kde.org/r/126996/#comment62841> this isn't identical. old code would match alt+shift+tab, yours doesn't. (tbh yours might be more right, but I want to check it's deliberate) - David Edmundson On Feb. 5, 2016, 8:49 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126996/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2016, 8:49 p.m.) > > > Review request for Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > On non-kdelibs4support depending modules. > > > Diffs > ----- > > applets/kicker/plugin/runnermodel.h d447efc > applets/kicker/plugin/runnermodel.cpp a3c193a > applets/pager/plugin/model.h 9382247 > applets/pager/plugin/model.cpp 1b80baa > applets/trash/plugin/dirmodel.h 9e7ad12 > applets/trash/plugin/dirmodel.cpp cc36e1a > containments/desktop/plugins/folder/mimetypesmodel.h f3d701c > containments/desktop/plugins/folder/mimetypesmodel.cpp 731414a > containments/desktop/plugins/folder/placesmodel.h b7f9f70 > containments/desktop/plugins/folder/placesmodel.cpp 9ca3bec > containments/desktop/plugins/folder/previewpluginsmodel.h d340eb5 > containments/desktop/plugins/folder/previewpluginsmodel.cpp 2b14551 > imports/activitymanager/switcherbackend.cpp 939575f > kcms/keyboard/kcm_view_models.h 434bbe9 > kcms/keyboard/preview/keysymhelper.cpp 1479fb0 > kcms/keyboard/tests/keyboard_daemon_test.cpp 885f070 > kcms/phonon/audiosetup.cpp bd6fcb0 > kcms/phonon/testspeakerwidget.cpp 45572be > kcms/solid_actions/ActionModel.cpp 36a55cf > kcms/solid_actions/PredicateModel.cpp 7d3a9da > kcms/solid_actions/SolidActions.cpp 9b607d9 > > Diff: https://git.reviewboard.kde.org/r/126996/diff/ > > > Testing > ------- > > Builds, tests pass. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel