D9121: [FrameSvgItem] Don't create unneccessary nodes

Kai Uwe Broulik noreply at phabricator.kde.org
Sun Dec 3 11:13:17 UTC 2017


broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  If a border isn't enabled there's no point in creating a node for it

TEST PLAN
  Borders still look fine also when they change. When enabled borders change we throw away the entire node anyway.

REPOSITORY
  R242 Plasma Framework (Library)

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

AFFECTED FILES
  src/declarativeimports/core/framesvgitem.cpp

To: broulik, #plasma
Cc: plasma-devel, #frameworks, 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/20171203/142405e9/attachment.html>


More information about the Plasma-devel mailing list