Re: Review Request: Drop Xinerama related options in KWin

2012-02-09 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103756/#review10459 --- This review has been submitted with commit 4a0369aae0ba400

Re: Review Request: Drop Xinerama related options in KWin

2012-01-25 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103756/ --- (Updated Jan. 26, 2012, 6:47 a.m.) Review request for kwin and Plasma. C

Re: Review Request: Drop Xinerama related options in KWin

2012-01-25 Thread Martin Gräßlin
> On Jan. 22, 2012, 7:54 p.m., Thomas Lübking wrote: > > kwin/geometry.cpp, line 250 > > > > > > should possibly be > > > > if (is_multihead) > >screen = screen_number; > > else if (screen == -1

Re: Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Thomas Lübking
> On Jan. 22, 2012, 6:09 p.m., Alex Fiestas wrote: > > Tested and everything works as expected :) ! > > Thomas Lübking wrote: > just ftr: afaics there's no way for a desktop wall (ie. treating all > screens as one) left, is it? > personally i don't mind, but there might have been claims

Re: Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103756/#review10006 --- kwin/geometry.cpp

Re: Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Alex Fiestas
> On Jan. 22, 2012, 6:09 p.m., Alex Fiestas wrote: > > Tested and everything works as expected :) ! > > Thomas Lübking wrote: > just ftr: afaics there's no way for a desktop wall (ie. treating all > screens as one) left, is it? > personally i don't mind, but there might have been claims

Re: Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Thomas Lübking
> On Jan. 22, 2012, 6:09 p.m., Alex Fiestas wrote: > > Tested and everything works as expected :) ! just ftr: afaics there's no way for a desktop wall (ie. treating all screens as one) left, is it? personally i don't mind, but there might have been claims on this at kcd. - Thomas --

Re: Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Alex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103756/#review1 --- Ship it! Tested and everything works as expected :) ! - Alex

Review Request: Drop Xinerama related options in KWin

2012-01-22 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103756/ --- Review request for kwin and Plasma. Description --- As discussed on t