D13748: Adjust Pager to libtaskmanager API changes for Wayland virtual desktop support
Marco Martin
noreply at phabricator.kde.org
Fri Sep 28 17:01:25 BST 2018
mart accepted this revision.
mart added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> pagermodel.cpp:444
> for (auto windowModel : d->windowModels) {
> - windowModel->setVirtualDesktop(0);
> + windowModel->setVirtualDesktop();
>
would it be possible to avoid default arguments?
> pagermodel.cpp:642
> + if (!winIds.isEmpty() && winIds.at(0).toUInt() == id) {
> + tasksModel->requestVirtualDesktops(idx, QVariantList() << itemId.toString());
> + break;
{} initializer?
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D13748
To: hein, mart
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180928/81d1f508/attachment.html>
More information about the Plasma-devel
mailing list