ngraham added a comment.
In D19036#412851 <https://phabricator.kde.org/D19036#412851>, @rooty wrote: > How's 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 > } > > implicitHeight: adjustHeight() > The lines below an `if` or `else` are always indented. 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