Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-06-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/ --- (Updated June 7, 2016, 1:23 p.m.) Status -- This change has been

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-06-06 Thread Marco Martin
> On June 5, 2016, 2:59 p.m., Anthony Fieroni wrote: > > Ping? the issues have not been adressed - Marco --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review96206

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-06-05 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review96206 --- Ping? - Anthony Fieroni On Май 12, 2016, 2:20 преди обяд,

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review95419 --- Fix it, then Ship it! logic is fine, but should not add

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/ --- (Updated May 12, 2016, 1:20 a.m.) Review request for KDE Frameworks,

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-10 Thread David Rosca
> On May 6, 2016, 4:08 p.m., David Rosca wrote: > > It will also need > > https://quickgit.kde.org/?p=plasma-workspace.git=commit=1b6231f2e3f1de8cb5ea2540f0b8f425f6ae43a3 > > in Plasma/5.6 > > Aleix Pol Gonzalez wrote: > No, it's already called by `ShellCorona::loadDefaultLayout()`. > >

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-10 Thread Aleix Pol Gonzalez
> On May 6, 2016, 6:08 p.m., David Rosca wrote: > > It will also need > > https://quickgit.kde.org/?p=plasma-workspace.git=commit=1b6231f2e3f1de8cb5ea2540f0b8f425f6ae43a3 > > in Plasma/5.6 > > Aleix Pol Gonzalez wrote: > No, it's already called by `ShellCorona::loadDefaultLayout()`. > >

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-10 Thread David Rosca
> On May 6, 2016, 4:08 p.m., David Rosca wrote: > > It will also need > > https://quickgit.kde.org/?p=plasma-workspace.git=commit=1b6231f2e3f1de8cb5ea2540f0b8f425f6ae43a3 > > in Plasma/5.6 > > Aleix Pol Gonzalez wrote: > No, it's already called by `ShellCorona::loadDefaultLayout()`. Yes,

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-06 Thread Aleix Pol Gonzalez
> On May 6, 2016, 6:08 p.m., David Rosca wrote: > > It will also need > > https://quickgit.kde.org/?p=plasma-workspace.git=commit=1b6231f2e3f1de8cb5ea2540f0b8f425f6ae43a3 > > in Plasma/5.6 No, it's already called by `ShellCorona::loadDefaultLayout()`. - Aleix

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-06 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review95241 --- It will also need

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-06 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review95240 --- src/plasma/corona.cpp (line 181)

Re: Review Request 127850: Let Plasma::Corona load the layout on all cases.

2016-05-06 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127850/#review95239 --- src/plasma/corona.cpp (line 181)