Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-10-25 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/#review7594 --- Ship it! mainwindow.cpp

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-10-21 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/ --- (Updated Oct. 21, 2011, 7:30 p.m.) Review request for Plasma. Changes --

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-10-05 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/#review7104 --- Ship it! Ship It! - Aaron J. Seigo On Sept. 24, 2011, 12:28

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-09-24 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/ --- (Updated Sept. 24, 2011, 12:28 p.m.) Review request for Plasma. Changes

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-09-15 Thread Aaron J. Seigo
> On Sept. 14, 2011, 10:53 a.m., Aaron J. Seigo wrote: > > editors/metadata/metadataeditor.cpp, lines 185-187 > > > > > > cd("..") only works if the project was created by plasmate. if it was > > imported from an e

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-09-15 Thread Antonis Tsiapaliokas
> On Sept. 14, 2011, 10:53 a.m., Aaron J. Seigo wrote: > > editors/metadata/metadata.ui, lines 312-314 > > > > > > wonder how that snuck in... my mystake. I guess that "LayoutDirectionAuto" is the correct option...

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-09-14 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/#review6497 --- great start, nearly there! some additional work is needed imho o