D28372: Added a merged look to the plasmoidheading and remove roundedborders

[Nathaniel Graham]

ngraham added a comment.


  So I have a heretical thought here: maybe we should consider using a CSD-style headerbar for system tray plasmoids with toolbar-style controls and put the controls on the same row as the header.
  
  No wait, hear me out! This pop-up doesn't suffer from most of the disadvantages of CSD headerbars <https://pointieststick.com/2018/12/18/on-headerbars/> since they aren't movable or resizable and we have total control over the presentation. And using this style would allow us to achieve visual consistency with Kirigami-style toolbars, which have the title on the left and a toolbar with controls on the right. We might need to increase the width of the pop-ups to accommodate long translations, and the space for toolbar content would be very limited so we'd need to use icons-only buttons for everything; no labels. The drawbacks might still outweigh the advantages. Anyway I just wanted to bring it up as an option. :) I'm not at all opposed to this idea here, I just wanted to present a possible alternative.

INLINE COMMENTS

> PlasmoidHeading.qml:58
> +     * If false, the plasmoidheading will display a separator. If true, it will be considered merged
> +     * with another plasmoidheading and the separator won't be showed. False by default.
> +     */

Might be worth saying, "...merged with another plasmoidheading underneath this one..." for clarity

REPOSITORY
  R242 Plasma Framework (Library)

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

To: niccolove, #plasma
Cc: ngraham, mart, davidedmundson, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200329/bc274289/attachment-0001.html>