davidedmundson requested changes to this revision. davidedmundson added a reviewer: davidedmundson. davidedmundson added a comment. This revision now requires changes to proceed.
Not your fault, but this current code is completely wrong and needs fixing. It should *not* be emitting a signal in setTitle. AppletInterface::setTitle calls Applet::setTitle -- which in turn emits Applet::titleChanged Applet::titleChanged is already connected to AppletInterface::titleChanged Tooltip is a bit more complex because there's some logic here, but in any case it needs to be following changes to Applet::title not AppletInterface::title REPOSITORY R242 Plasma Frameworks REVISION DETAIL https://phabricator.kde.org/D3758 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, davidedmundson Cc: davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas