----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75004 -----------------------------------------------------------
dataengines/notifications/notificationsengine.cpp <https://git.reviewboard.kde.org/r/122314/#comment51936> what about if you had \n \n \n I think this will still leave one. Maybe <br/>(\s*|<br/>)* - David Edmundson On Jan. 29, 2015, 7:14 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122314/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2015, 7:14 p.m.) > > > Review request for Plasma. > > > Bugs: 343532 > https://bugs.kde.org/show_bug.cgi?id=343532 > > > Repository: plasma-workspace > > > Description > ------- > > While looking into https://bugs.kde.org/show_bug.cgi?id=343532 I noticed that > wall sends message always starting with \n, but with our limited space, that > makes no sense. > > With this patch the text is first trimmed from whitespace at the beginning > and end, then all \ns are converted to <br/> (as before), then the text > replaces all remaining inner whitespace with single space and finally, if > there are two new lines in succession, it turns it into just one. > > The bug itself, which is about messages from wall not appearing in the popup, > will only partially get solved by this as the popup is limited in max height, > so it displays at most 3 lines of text and with wall first two lines are > already taken, that leaves just one line for the actual message. > > However these are persistant so they get saved in the notification history > anyway. > > > Diffs > ----- > > dataengines/notifications/notificationsengine.cpp 6e566b4 > > Diff: https://git.reviewboard.kde.org/r/122314/diff/ > > > Testing > ------- > > Now I'm able to see the message sent with wall. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
