---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1028/#review1833
---
Ship it!
Patch looks fine, other than some minor coding style
On Friday 17 July 2009 23:06:56 Sebastian Kügler wrote:
On Friday 17 July 2009 18:25:06 Arthur Renato Mello wrote:
On Fri, Jul 17, 2009 at 9:10 AM, Sebastian Küglerse...@kde.org wrote:
However, I have rather substantial changes to the frame applet on my
disk. I'm resolving some issues
On Tuesday 14 July 2009 19:36:08 Arthur Mello wrote:
As mentioned on Frame TODO this patch adds buttons to navigate through
slide show. Buttons appear when mouse is over applet and only when applet
is doing a slideshow. Example code at TODO put the buttons above the
pictue, I placed them on
On Friday 17 July 2009 18:25:06 Arthur Renato Mello wrote:
On Fri, Jul 17, 2009 at 9:10 AM, Sebastian Küglerse...@kde.org wrote:
However, I have rather substantial changes to the frame applet on my
disk. I'm resolving some issues that I wouldn't like to see committed
this weekend and am
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1028/
---
Review request for Plasma.
Summary
---
As mentioned on Frame TODO this
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1028/#review1601
---
Works well and I don't see anything wrong in code on a quick look.
On 2009-07-14 21:04:59, Anne-Marie Mahfouf wrote:
Works well and I don't see anything wrong in code on a quick look. Sebas,
can you take a quick look as you'll add remote URL support?
Thanks Arthur for this patch!
A pause button was also in the wish list... ;)
Referring to
