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