Re: Review Request 122314: Sanitize whitespace in notification popups

2015-02-05 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- (Updated Feb. 5, 2015, 10:15 a.m.) Status -- This change has been ma

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-02-04 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75400 --- Ship it! Ship It! - David Edmundson On Jan. 30, 2015, 11:3

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-02-02 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75248 --- bump - Martin Klapetek On Jan. 30, 2015, 12:37 p.m., Martin

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-01-30 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/ --- (Updated Jan. 30, 2015, 12:37 p.m.) Review request for Plasma. Changes

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-01-29 Thread Martin Klapetek
> On Jan. 29, 2015, 10:31 p.m., David Edmundson wrote: > > dataengines/notifications/notificationsengine.cpp, line 229 > > > > > > what about if you had > > > > \n\n\n > > > > I think th

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-01-29 Thread David Edmundson
> On Jan. 29, 2015, 9:31 p.m., David Edmundson wrote: > > dataengines/notifications/notificationsengine.cpp, line 229 > > > > > > what about if you had > > > > \n\n\n > > > > I think thi

Re: Review Request 122314: Sanitize whitespace in notification popups

2015-01-29 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122314/#review75004 --- dataengines/notifications/notificationsengine.cpp