filipf added a comment.
In D20144#442451 <https://phabricator.kde.org/D20144#442451>, @GB_2 wrote: > In D20144#441802 <https://phabricator.kde.org/D20144#441802>, @ngraham wrote: > > > Hmm, the separator line feels like a bit much to me. How about just `units.largeSpacing` or `units.gridUnit` between them? > > > This is it without separator, I personally think it looks weird with that gap: F6746651: More Obvious Location of Lock Widgets and Remove Panel Buttons (Vertical) (No Separator).png <https://phabricator.kde.org/F6746651> > Let's see what others think. +1 for the idea of the patch Gap - looks weird to me as well Separator - it's okay IMO If we really wanted to be sure we could have a confirmation dialog for deleting the panel, although the Undo button you get in a notification somewhat lessens the need for it. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D20144 To: GB_2, #plasma, #vdg Cc: ndavis, abetts, filipf, rooty, ngraham, #vdg, plasma-devel, #plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
