Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-04 Thread Andreas Kainz
> On Juli 2, 2016, 11:06 nachm., David Edmundson wrote: > > Requires two tiny changes in the cpp, which I've done. > > > > Are you happy for me to push with the compilation fixes? > > Andreas Kainz wrote: > so you already ship the request? or the change cpp file? > > David Edmundson wrote:

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-04 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/ --- (Updated July 4, 2016, 11:40 p.m.) Status -- This change has been ma

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-03 Thread Andreas Kainz
> On Juli 2, 2016, 11:06 nachm., David Edmundson wrote: > > Requires two tiny changes in the cpp, which I've done. > > > > Are you happy for me to push with the compilation fixes? > > Andreas Kainz wrote: > so you already ship the request? or the change cpp file? > > David Edmundson wrote:

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-02 Thread David Edmundson
> On July 2, 2016, 11:06 p.m., David Edmundson wrote: > > Requires two tiny changes in the cpp, which I've done. > > > > Are you happy for me to push with the compilation fixes? > > Andreas Kainz wrote: > so you already ship the request? or the change cpp file? I applied your patch, then I

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-02 Thread Andreas Kainz
> On Juli 2, 2016, 11:06 nachm., David Edmundson wrote: > > Requires two tiny changes in the cpp, which I've done. > > > > Are you happy for me to push with the compilation fixes? so you already ship the request? or the change cpp file? - Andreas

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-02 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/#review97041 --- Ship it! Requires two tiny changes in the cpp, which I've

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-07-02 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/ --- (Updated Juli 2, 2016, 5:20 nachm.) Review request for Plasma. Changes

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-06-28 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/ --- (Updated Juni 28, 2016, 9:12 nachm.) Review request for Plasma. Reposit

Re: Review Request 128310: standardize kcm layout for cursortheme

2016-06-28 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/ --- (Updated Juni 28, 2016, 8:59 nachm.) Review request for Plasma. Changes

Review Request 128310: standardize kcm layout for cursortheme

2016-06-28 Thread Andreas Kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128310/ --- Review request for Plasma. Repository: plasma-desktop Description -