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: | ||
Sentry Crash Report: | |||
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
Created attachment 22927 [details]
How it looked before the new folder was created
Created attachment 22928 [details]
The bug, here I created the TEST folder
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 Hi András, good news: I could fix this issue and will commit an update this evening :-) 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 |