[Differential] [Accepted] D1009: Added indicator of whether there are windows in an activity to the switcher

2016-02-22 Thread davidedmundson (David Edmundson)
davidedmundson accepted this revision.
davidedmundson added a comment.


  +1 too

INLINE COMMENTS
  imports/activitymanager/sortedactivitiesmodel.h:106 worth #ifdef'ing this.
  
  as that way future users will get an error if they try and use it rather than 
an uninitialised var.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

BRANCH
  master

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

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

To: ivan, graesslin, sebas, mart, davidedmundson
Cc: plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Differential] [Accepted] D1009: Added indicator of whether there are windows in an activity to the switcher

2016-02-22 Thread mart (Marco Martin)
mart accepted this revision.
mart added a comment.
This revision is now accepted and ready to land.


  it will take a lot of thinking on the wayland side ;) but yes, i like it

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

BRANCH
  master

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

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

To: ivan, davidedmundson, graesslin, sebas, mart
Cc: plasma-devel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel