rooty added a comment.
In D19036#412890 <https://phabricator.kde.org/D19036#412890>, @ngraham wrote: > Here, like this: > > function adjustHeight() > { > var layoutHeight = mainLayout.height > if (bodyText.lineCount > 1) { > layoutHeight = mainLayout.height > } else { > if (appIconItem.valid || imageItem.nativeWidth > 0) { > layoutHeight = Math.max((mainLayout.height + 1.5 * units.smallSpacing),(units.iconSizes.large + 2 * units.smallSpacing)) > } else { > if (bottomPart.height != 0) { > if (mainLayout.height > units.iconSizes.large) { > layoutHeight = mainLayout.height + 1.5 * units.smallspacing > } else { > layoutHeight = mainLayout.height + 2 * units.smallSpacing > } > } else { > layoutHeight = mainLayout.height + units.smallSpacing // because of the close button > } > } > } > return layoutHeight > } > Ooh that does look nicer... REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D19036 To: rooty, #plasma, #vdg Cc: filipf, ngraham, abetts, broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
