Review Request 113939: Update the geometries of the viewport and the scene of KItemListContainer with the size of the view since the scrollbar visibility may have changed.

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113939/#review44081
-----------------------------------------------------------


This review has been submitted with commit b3322111c1dd58723b10d2308bf790e61fc9523b by Frank Reininghaus on behalf of Yichao Yu to branch KDE/4.11.

- Commit Hook


On Nov. 19, 2013, 5:05 p.m., Yichao Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113939/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2013, 5:05 p.m.)
> 
> 
> Review request for Dolphin, Frank Reininghaus and Peter Penz.
> 
> 
> Bugs: 327709
>     http://bugs.kde.org/show_bug.cgi?id=327709
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> When resizing the window and when `KItemListContainer::updateGeometries` is called before the scrollbar visibility is updated, a relayout is triggered in `m_controller->view()->setGeometry` which updates the scrollbar visibility and calls back to `KItemListContainer::updateGeometries` again. Since the first call, which has the wrong geometry (due to the incorrect scrollbar states), updates the geometries of the scene and viewport after the second call (which has the right geometry!!) returns, the final result is a size corresponds to old scrollbar state.
> 
> This patch uses the new geometry of the view after updating it (since it might not be the size we put in) and therefore makes the sizes consistent.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/kitemviews/kitemlistcontainer.cpp cd2fca1 
> 
> Diff: http://git.reviewboard.kde.org/r/113939/diff/
> 
> 
> Testing
> -------
> 
> Compiles, solves the problem here.
> 
> 
> Thanks,
> 
> Yichao Yu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20131120/628f56af/attachment.htm>