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

[Krešimir Čohar]

rooty added a comment.


  In D19036#412830 <https://phabricator.kde.org/D19036#412830>, @ngraham wrote:
  
  > Can you use the typical KDE coding style for if/else blocks?
  
  
  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()

REPOSITORY
  R120 Plasma Workspace

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

To: rooty, #plasma, #vdg
Cc: ngraham, abetts, broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, 
ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart