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