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

[Krešimir Čohar]

rooty added a comment.


  In D19036#412890 <https://phabricator.kde.org/D19036#412890>, @ngraham wrote:
  
  > 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
  >   }
  >
  
  
  Ooh that does look nicer...

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/918e3542/attachment.html>