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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190215/6391f125/attachment.html>