[Differential] [Requested Changes To] D3758: [AppletInterface] Emit toolTipMainTextChanged if it changes in response to a title change

davidedmundson (David Edmundson) noreply at phabricator.kde.org
Tue Dec 20 12:03:12 UTC 2016


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


More information about the Plasma-devel mailing list