[Differential] [Commented On] D2218: New logic for screen numbers in plasmashell

mart (Marco Martin) noreply at phabricator.kde.org
Fri Jul 22 09:26:18 UTC 2016


mart added a comment.


  In https://phabricator.kde.org/D2218#41871, @davidedmundson wrote:
  
  > Personally, I think the part that's out of place is ScreenPool touching the desktop views; if you move m_desktopViewforId to shellCorona the design all fits:
  >  insert/remove in ScreenPool become reflective
  >  and all DesktopView stuff is within one class, screen to ID mapping is in one class.
  
  
  I tried to move the panels logic in screenpool but i didn't really like it, screenpool kindof became almost a small clone of corona..
  I'll try instead moving m_desktopViewforId in corona

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, #plasma
Cc: davidedmundson, rwooninck, graesslin, plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160722/47ee435e/attachment-0001.html>


More information about the Plasma-devel mailing list