niccolove added a comment.

  I used the Svg instead of the plasmoidHeading to avoid using a Control. 
User-wise, it works correctly, but can I get your opinion code-wise?

INLINE COMMENTS

> broulik wrote in FullRepresentation.qml:454
> Move the `Svg` somewhere outside the delegate, I don't want multiple 
> instances of it

I'm a bit confused here. The PlasmaCore.svg{id: lineSvg...} element was inside 
a different PlasmaCore.SvgItem that I removed. So what I did was essentially to 
move it from that SvgItem to this SvgItem. That one is the only instance of 
PlasmaCore.Svg.

> broulik wrote in NotificationItem.qml:125
> hm, ok

I moved it in a RowLayout to remove the necessity of checking altogether 
(otherwise, I should've put anchors.margins AND layout.margins for each case 
and it would've have been quite ugly imo)

REPOSITORY
  R120 Plasma Workspace

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

To: niccolove, #plasma, broulik
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to