Review Request 126996: Drop usage of Qt4Support
Aleix Pol Gonzalez
aleixpol at kde.org
Thu Feb 11 16:44:25 UTC 2016
> On Feb. 6, 2016, 12:22 a.m., David Edmundson wrote:
> > imports/activitymanager/switcherbackend.cpp, line 125
> > <https://git.reviewboard.kde.org/r/126996/diff/1/?file=442834#file442834line125>
> >
> > 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)
>
> Aleix Pol Gonzalez wrote:
> I'm not sure, I'll add Ivan.
Dropping the issue since Ivan ack'ed it.
- Aleix
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126996/#review92109
-----------------------------------------------------------
On Feb. 8, 2016, 3:03 a.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126996/
> -----------------------------------------------------------
>
> (Updated Feb. 8, 2016, 3:03 a.m.)
>
>
> Review request for Plasma and Ivan Čukić.
>
>
> 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/desktoptheme/thememodel.cpp 8274b72
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160211/cb0c4ea1/attachment-0001.html>
More information about the Plasma-devel
mailing list