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

[Krešimir Čohar]

rooty added a comment.


  In D19036#412876 <https://phabricator.kde.org/D19036#412876>, @ngraham wrote:
  
  > The lines below an `if` or `else` are always indented.
  
  
  
  
    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: 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/5b3645cb/attachment.html>