D20510: SvgItem, IconItem: drop "smooth" property override, update node on change

[Friedrich W. H. Kossebau]

kossebau added a comment.


  A good example to see how the smooth property was broken is the animation of the analog clock's hands (best enable second hand to see often):
  the Hand.qml has "smooth: !anim.running" set as property for the SvgItem. Without this patch, the hand during animation looks fine. With this patch, the hand looks very unsmooth during animation.
  
  As aftermath I would actually do a patch to change this to "smooth: true", given it looks quite bad and no-one before complained about resource issues, so this once intended optimization seems unneeded now.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: kossebau, #plasma, davidedmundson, mart
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190414/dbf66b02/attachment.html>