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

[Krešimir Čohar]

rooty marked 3 inline comments as done.
rooty added a comment.


  I hope I didn't mess up the syntax...
  
    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()
  
  
  
  In D19036#412788 <https://phabricator.kde.org/D19036#412788>, @abetts wrote:
  
  > +1
  
  
  Thanks! :D

REPOSITORY
  R120 Plasma Workspace

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

To: rooty, #plasma, #vdg
Cc: 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/2a2361c0/attachment.html>