D19036: [notifications] Lift up notification content if one line of body text droops

[Nathaniel Graham]

ngraham added a comment.


  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
    }

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