Re: Review Request 114741: Add time duration to media item for youtube videos.

2013-12-30 Thread Sujith Haridasan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114741/#review46472 --- First of all a big thanks to Sudhendhu for the effort. Good jo

Re: Review Request 114741: Add time duration to media item for youtube videos.

2013-12-30 Thread Sudhendu Kumar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114741/ --- (Updated Dec. 31, 2013, 6:46 a.m.) Review request for Plasma, Shantanu Tu

Re: Review Request 114741: Add time duration to media item for youtube videos.

2013-12-30 Thread Shantanu Tushar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114741/#review46470 --- Works nicely :) Just one nitpick around the hh:mm:ss formattin

Re: Review Request 114746: [calculator runner] Replace regexp with explicit character category check

2013-12-30 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114746/ --- (Updated Dec. 31, 2013, 3:06 a.m.) Review request for Plasma. Changes -

Review Request 114746: [calculator runner] Replace regexp with explicit character category check

2013-12-30 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114746/ --- Review request for Plasma. Bugs: 321803 http://bugs.kde.org/show_bug.

Review Request 114741: Add time duration to media item for youtube videos.

2013-12-30 Thread Sudhendu Kumar
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114741/ --- Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith Harida

Re: Students to work on Plasma

2013-12-30 Thread Bhushan Shah
Hello! On Mon, Dec 30, 2013 at 3:37 PM, Marco Martin wrote: > porting away from kde4support > http://community.kde.org/Open_Academy#6._Get_rid_of_KDE4Support_dependecies_in_Plasma2 I am afraid that it is even task! Quick git grep, CMakeLists.txt:Bookm

Re: Setup build of kdeplasma-addons on CI system

2013-12-30 Thread David Edmundson
On Mon, Dec 30, 2013 at 8:04 AM, Bhushan Shah wrote: > Hello! > > On Sun, Dec 29, 2013 at 6:49 PM, Sebastian Kügler wrote: >> So please everybody, tackle bugs, do not port code that isn't really needed >> for a basic functional system. It might be more fun, or maybe more rewarding >> to pick the

Re: Students to work on Plasma

2013-12-30 Thread Artur Souza
On Mon, Dec 30, 2013 at 8:07 AM, Marco Martin wrote: > > So, i added there two tasks: > porting away from kde4support > http://community.kde.org/Open_Academy#6._Get_rid_of_KDE4Support_dependecies_in_Plasma2 > > and the adapting of plasmoids to api change > http://community.kde.org/Open_Academy#7._

Re: Students to work on Plasma

2013-12-30 Thread Marco Martin
On Monday 30 December 2013, Artur Souza wrote: > Hey Marco! :D > > On Mon, Dec 30, 2013 at 6:56 AM, Marco Martin wrote: > > ehi Arthur, such a long time :D > > Indeed :) > > > a possible painful, but not hard idea: > > I'm not sure if we are happy about the api of qml bindings, well, i am > > n

Re: Students to work on Plasma

2013-12-30 Thread Artur Souza
Hey Marco! :D On Mon, Dec 30, 2013 at 6:56 AM, Marco Martin wrote: > > ehi Arthur, such a long time :D Indeed :) > a possible painful, but not hard idea: > I'm not sure if we are happy about the api of qml bindings, well, i am not, > but is a matter of how much pain causes to change now. > if w

Re: Students to work on Plasma

2013-12-30 Thread Marco Martin
On Friday 27 December 2013, Artur Souza wrote: > Hello my friends! ehi Arthur, such a long time :D > You probably have seen my post on planet.kde > (http://blog.morpheuz.cc/26/12/2013/open-academy-and-kde/) about > OpenAcademy and the possibility of sponsoring students to work on KDE. > > I woul