mart reopened this revision.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> main.qml:31
>  
> -    Layout.minimumWidth: units.iconSizes.small
> -    Layout.minimumHeight: units.iconSizes.small
> +    readonly property bool inPanel: (plasmoid.location == 
> PlasmaCore.Types.TopEdge
> +        || plasmoid.location == PlasmaCore.Types.RightEdge

to  detect if this is in a panel was intended from the beginning the 
formfactor, not the location

> main.qml:37
> +    Layout.maximumWidth: inPanel ? units.iconSizeHints.panel : -1
> +    Layout.maximumHeight: inPanel ? units.iconSizeHints.panel : -1
>  

this will break fitts law on an horizontal panel, as mouse wouldn't work on 
screen edge?

REPOSITORY
  R119 Plasma Desktop

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

To: hein, #plasma, mart, broulik
Cc: mvourlakos, ivan, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

Reply via email to