Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/ --- (Updated Oct. 11, 2016, 1:33 p.m.) Status -- This change has been

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99935 --- Ship it! Ship It! - Marco Martin On Oct. 10, 2016,

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
> On Oct. 11, 2016, 8:52 a.m., Kai Uwe Broulik wrote: > > I have a FolderView applet in my panel (pointing to Downloads folder) and > > it is blank with this patch now. ok, so please don't push this yet - Marco --- This is an

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99923 --- I have a FolderView applet in my panel (pointing to

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99921 --- Ship it! this version seems to work well - Marco Martin

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/ --- (Updated Oct. 10, 2016, 11:48 p.m.) Review request for KDE Frameworks

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99917 --- I confirm Kai's findings, to me it breaks loading of the

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99914 --- Ship it! Ship It! - Marco Martin On Oct. 10, 2016,