Bug 155378

Summary:	Icons are messed up after creating a new folder
Product:	[Applications] dolphin	Reporter:	András Manţia <amantia>
Component:	general	Assignee:	Peter Penz <peter.penz19>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	16.12.2
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	How it looked before the new folder was created The bug, here I created the TEST folder

Description András Manţia 2008-01-10 08:08:17 UTC

Version:            (using KDE Devel)
Installed from:    Compiled sources

If I have a folder with many entries and there I create a new folder, the icon layout is messed up completely. The strange is:
- it happened (always) in Konqueror using the dolphin part (see the screenshots)
- I couldn't reproduce in Dolphin
- after Dolphin was started I couldn't reproduce in Konqueror

Comment 1 András Manţia 2008-01-10 08:08:59 UTC

Created attachment 22927 [details]
How it looked before the new folder was created

Comment 2 András Manţia 2008-01-10 08:10:26 UTC

Created attachment 22928 [details]
The bug, here I created the TEST folder

Comment 3 Peter Penz 2008-01-10 16:45:28 UTC

Hi András! This is a known issue and seems to be inside Qt 4.3.x. But I'm not 100 % sure yet and need time reproducing this within a Qt-only application to prove it. At the moment Dolphin bypasses this issue by a workaround, but this workaround might fail too with very long filenames (without the workaround even more items would be aligned in a wrong manner). It's on my internal TODO list and this issue will be fixed, I just fear that I need some time for this and I'm not sure whether I can fix this until 4.0.1

Comment 4 Peter Penz 2008-03-20 08:43:20 UTC

Hi András, good news: I could fix this issue and will commit an update this evening :-)

Comment 5 Peter Penz 2008-03-20 17:59:28 UTC

SVN commit 788095 by ppenz:

QListView does not support having a margin for grids. Originally it has been tried to bypass this by overwriting QListView::visualRect(), but this has some side effects (see #155378 and #155575). The clean approach is to return a proper size hint in the file item delegate.

Currently a custom item delegate has been made for Dolphin, but we'll discuss whether it makes sense providing this feature already in KFileItemDelegate...

BUG: 155378
BUG: 155575
CCMAIL: fredrik@kde.org

 M  +1 -0      CMakeLists.txt  
 A             dolphinfileitemdelegate.cpp   [License: GPL (v2+)]
 A             dolphinfileitemdelegate.h   [License: GPL (v2+)]
 M  +6 -44     dolphiniconsview.cpp  
 M  +0 -3      dolphiniconsview.h  
 M  +2 -2      dolphinview.cpp  
 M  +2 -1      dolphinview.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=788095