D27182: [applets/taskmanager] Clean up Tooltip code

[Tranter Madi]

trmdi added inline comments.

INLINE COMMENTS

> ToolTipInstance.qml:39
>  
> -    spacing: units.smallSpacing
> -

Why do you remove all `spacing`? This increases lots of wasted space which is not good at least on small screens like 1366x768.

> ToolTipInstance.qml:79
> +        // This number controls the overall size of the window tooltips
> +        Layout.maximumWidth: units.gridUnit * 17
> +        Layout.minimumWidth: isWin ? Layout.maximumWidth : 0

Should we make the height `const` like this too? e.g. make it a golden rectangle?
F8088397: image.png <https://phabricator.kde.org/F8088397>

Otherwise, the size of the tooltip may be not consistent in all cases, e.g. a normal window vs a window which is available on all activities.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  clean-up-tooltip-code (branched from master)

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

To: ngraham, #plasma, #vdg, ndavis
Cc: trmdi, broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200206/e3c0b74e/attachment.html>