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