Version: 1.99 (using Devel) OS: Linux It seems that dolphin's scroll area just think the last row of Icon has only one line of text in file name. Reproducible: Sometimes Steps to Reproduce: 1. Open a folder with a lot of file that need a scroll bar 2. Scroll to the bottom Actual Results: Last row can be shown completely Expected Results: Last row can be shown with only one line text. It's only happen in some case but not others..
Created attachment 65918 [details] Screenshot
Thanks for the bug report! I can confirm this in master. Note that it does show the full names for all items in the last row if the very last file in the view is the one with the longest name. The problem seems to be that KItemListViewLayouter::doLayout() sets m_maximumScrollOffset = m_itemRects.last().bottom(), which is not necessarily the real maximum scroll offset. It should calculate the maximum of the bottom() for all itemRects in the last row.
Created attachment 66389 [details] Patch This patch fixes it for me, but it might go wrong if grouping is enabled...
Created attachment 66390 [details] The right patch Sorry, I clicked the wrong patch file. This should be the right patch now, but as I said, I'm not sure if it works well with grouping.
Thanks Frank for the patch! As you guessed there will be minor issues on some corner-cases when grouping is turned on: E.g. assuming that each group only contains one item (except the last that contains e.g. 2) although we have 5 columns then 'index' will be too large. I've an idea how to fix this based on your patch and will test + push it ~tomorrow :-)
Git commit b1d75b49e2dea4a91a76be6d9e9251bba6826d36 by Peter Penz. Committed on 05/12/2011 at 19:56. Pushed by ppenz into branch 'master'. Fix calculation of maximum scroll offset The maximum scroll offset must be the maximum y-range of the last row, not only the maximum of the last item. Thanks to Frank Reininghaus for analysing the root-cause and for providing the initial patch! BUG: 287211 FIXED-IN: 4.8.0 M +9 -0 dolphin/src/kitemviews/kitemlistviewlayouter.cpp http://commits.kde.org/kde-baseapps/b1d75b49e2dea4a91a76be6d9e9251bba6826d36