Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-04-05 Thread nucleo nucleo
> On March 31, 2013, 10:19 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 81fd37ec62cf8f064996aac409e6808ec7916153 by David Edmundson to branch > > KDE/4.10. > > nucleo nucleo wrote: > I still see popups with things like "" with kde-workspace-4.10.2 and > back

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-04-05 Thread nucleo nucleo
> On March 31, 2013, 10:19 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 81fd37ec62cf8f064996aac409e6808ec7916153 by David Edmundson to branch > > KDE/4.10. > > nucleo nucleo wrote: > I still see popups with things like "" with kde-workspace-4.10.2 and > back

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-04-05 Thread nucleo nucleo
> On March 31, 2013, 10:19 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 81fd37ec62cf8f064996aac409e6808ec7916153 by David Edmundson to branch > > KDE/4.10. I still see popups with things like "" with kde-workspace-4.10.2 and backported patch. I use DBus based po

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-04-04 Thread David Edmundson
> On March 31, 2013, 10:19 p.m., Commit Hook wrote: > > This review has been submitted with commit > > 81fd37ec62cf8f064996aac409e6808ec7916153 by David Edmundson to branch > > KDE/4.10. > > nucleo nucleo wrote: > I still see popups with things like "" with kde-workspace-4.10.2 and > back

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-04-01 Thread nucleo nucleo
> On March 20, 2013, 2:20 p.m., Aaron J. Seigo wrote: > > I'm ok with this, though I have two reservations: > > > > * other notification systems do not support rich text > > * we should probably define a subset of rich text that is guaranteed to be > > supported This patch will be applied in 4

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-03-31 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109492/ --- (Updated March 31, 2013, 10:19 p.m.) Status -- This change has been m

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-03-31 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109492/#review30191 --- This review has been submitted with commit 81fd37ec62cf8f06499

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-03-20 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109492/#review29558 --- Ship it! I'm ok with this, though I have two reservations: *

Re: Review Request 109492: Allow Rich Text in QML notifications plasmoid

2013-03-17 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109492/ --- (Updated March 17, 2013, 10:22 p.m.) Review request for kde-workspace, Pla