Re: Review Request: again on webview+scrollwidget

2010-03-27 Thread Marco Martin
> On 2010-03-26 02:38:40, Zack Rusin wrote: > > Looking good, thanks for taking it all the way! The two whitespace changes > > should be probably removed. In my initial patch I had that setDragToScroll > > changed from false to true in webview, I'm not sure if we want to keep > > that. Does it

Re: Review Request: again on webview+scrollwidget

2010-03-25 Thread Zack Rusin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3399/#review4677 --- Ship it! Looking good, thanks for taking it all the way! The two whit

Re: Review Request: again on webview+scrollwidget

2010-03-25 Thread Zack Rusin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3399/#review4678 --- Ship it! Looking good, thanks for taking it all the way! The two whit

Review Request: again on webview+scrollwidget

2010-03-25 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3399/ --- Review request for Plasma. Summary --- this is 99% the zack's patch he sent