D10315: [Notifications] Fix grouping

Fabian Vogt noreply at phabricator.kde.org
Mon Feb 5 08:54:50 UTC 2018


fvogt added inline comments.

INLINE COMMENTS

> notificationsengine.cpp:230
> +            if (previousBody != bodyFinal) {
> +                bodyFinal = previousBody + QStringLiteral("<br/>") + bodyFinal;
>              }

This results in `...Text<br/><?xml version="1.0"?><html>Some text...` AFAICT.

This also means that the if condition is always false.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, davidedmundson, aacid, fvogt
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180205/d90045d9/attachment.html>


More information about the Plasma-devel mailing list