D20144: Make location of "Lock Widgets" and "Remove Panel" buttons more obvious
Filip Fila
noreply at phabricator.kde.org
Wed Apr 3 15:48:03 BST 2019
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190403/d9c21025/attachment.html>
More information about the Plasma-devel
mailing list