D8567: Return the screen id for a screen name.

Milian Wolff noreply at phabricator.kde.org
Wed Nov 1 10:42:00 UTC 2017


mwolff added inline comments.

INLINE COMMENTS

> shellcorona.cpp:1162
>                  removeDesktop(v);
> +                emit screenRemoved(m_screenPool->id(screen->name()));
> +            }

shouldn't this be done from within `removeDesktop`? considering this duplicates the change above?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D8567

To: amantia, #plasma, ervin, dvratil, mlaurent, hein
Cc: mwolff, apol, mart, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171101/b048569d/attachment.html>


More information about the Plasma-devel mailing list