D10516: Update status to NeedsAttention when there is a menu

Martin Kostolný noreply at phabricator.kde.org
Wed Feb 14 21:23:35 UTC 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180214/b6311bfd/attachment-0001.html>


More information about the Plasma-devel mailing list