martinkostolny accepted this revision.
martinkostolny added a comment.
This revision is now accepted and ready to land.


  Thanks for fixing my code! :)
  
  I'm sure I can also provide a flag for menu availability (the code for 
appmenu is copied for now anyway). But I think in AWC it does not make sense to 
set status to active when menu is available because it has other use then 
showing menu. I can be wrong of course.
  
  Anyway I definitely agree with your proposed patch.

REPOSITORY
  R884 Active Window Control Applet for Plasma

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

To: mvourlakos, martinkostolny
Cc: broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to