Version: 1.4 (using KDE 4.4.2) Compiler: cc OS: Linux Installed from: Ubuntu Packages Open Dolphin, choose View->View Mode->Details, select a file (or a picture file), and use the zoom to enlarge the views. The previews grow, but the rows height doesn't, and so the pictures previews overlap, generating a mess of previews. You can now change to Icons view and come back to Details view. If no file is selected, zoom on previews works fine. But selecting a file, it works wrong again. Using the zoom to reduce preview always works fine.
Thanks for the bug report! I can confirm this problem in current trunk.
I found the same problem. If you refresh the page the rows height would be ok. So I think the issue is that rows aren't refreshed when you use the slider to zoom-in
I don't think the problem is that rows aren't refreshed. Rows do actually change its size when moving the slider, if you make previews smaller and bigger again. Make icons size to 128 pixels, for example. Then move down and up the preview size slider. The rows change size, up to a maximum to fit the 128 pixels preview. It also has some estrange behaviour when the scroll bars appear of disappear.
Dolphin 1.5 with KDE 4.5.1 still shows the problem
Yes, you're right. It's still reproducible in trunk as well. I think it might be related to the recently reported bug 250017.
I've investigated into this issue already and it looks like a bug in QTreeView. However I did not find the time yet to confirm this to 100 % :-(
I'm using the QTreeView with detailed view mode (with QT4.7.0, KDE 4.5.1) for other projects, this row resizing error does NOT happen when changing the icon size. I use the property treeview->setIconSize(QSize(value, value)); And it works perfectly. The rows are resized as needed. However, it does not work correctly with my Dolphin (version 1.5) neither Konqueror (v 4.5.1). On the other hand, a college of me has kde 4.3.2, Dolphin 1.3 and QT 4.5.2, and row resizing works correctly both for Dolphin and the QTreeView objects we use in our projects.
*** Bug 250017 has been marked as a duplicate of this bug. ***
Created attachment 52223 [details] Suggested fix I'm afraid it's my fault. This is a regression (since 4.4.1) caused by my fix for bug 226666. Sorry to everyone for the trouble. @Peter: could you have a look at my suggested changes to DolphinDetailsView::updateDecorationSize(...) and DolphinDetailsView::adjustMaximumSizeForEditing(...)? Just to make sure that I don't cause another regression ;-)
Created attachment 52224 [details] Alternative fix Maybe this fix is a bit more elegant because it does not set the height of maximumSize "by hand".
*** Bug 253289 has been marked as a duplicate of this bug. ***
Thanks Frank for investigating into this! I think the "Alternate fix" looks fine, please commit :-)
SVN commit 1182797 by freininghaus: Prevent that icons overlap in Details View when zooming. The problem was that a maximum size was assigned to KFileItemDelegate for displaying items without considering that icon zooming may change the item height. CCBUG: 234600 M +12 -0 dolphindetailsview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1182797
SVN commit 1182798 by freininghaus: Prevent that icons overlap in Details View when zooming. The problem was that a maximum size was assigned to KFileItemDelegate for displaying items without considering that icon zooming may change the item height. BUG: 234600 FIXED-IN: 4.5.3 M +12 -0 dolphindetailsview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1182798
*** Bug 259399 has been marked as a duplicate of this bug. ***