Bug 251461

Summary:	dolphin information panel shows up wrong icon
Product:	[Unmaintained] kio	Reporter:	Aaron Lewis <the.warl0ck.1989>
Component:	general	Assignee:	David Faure <faure>
Status:	RESOLVED FIXED
Severity:	minor	CC:	adrian, faure, frank78ac, nate
Priority:	LO
Version First Reported In:	4.10.5
Target Milestone:	---
Platform:	Chakra
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Aaron Lewis 2010-09-16 16:25:12 UTC

Version:           unspecified (using KDE 4.5.1) 
OS:                Linux

I've got a directory , say it /Library , and /Library/.directory sets directory icon to folder-book , and also i've got a bookmark ( shows up everytime in a dialog , something similar to QFileDialog ) named /Library , pointing to same directory , but icon was set to folder-book2

And thus , when dolpin opened /Library , information panel will show up folder-book2 , which is definitely wrong , i'm wondering why it reads from bookmarks rather than .directory file.

Thanks.

Reproducible: Always

Steps to Reproduce:
Create a directory , setting icon to folder-a ;
In any KDE Dialog used to open files , save files or whatever that with a bookmark menu on the left , add an bookmark , pointing to same directory , but icon set to another one.

Enable information panel in dolpin , move to that directory , see the preview icon of that directory is with wrong icon.

Comment 1 Aaron Lewis 2010-09-16 16:26:20 UTC

Another clew is when you set the right icon on bookmark , preview shows up right.

Comment 2 Matthias Fuchs 2011-06-03 01:17:02 UTC

This is not really a dolphin problem.
Dolphin at this point uses KFileItem::pixmap [1], so the "bug" rather is in 
that code.

Unrelated to this bug I wonder why KFileItem::pixmap does not use 
KFileItem::iconName internally.

PS.: I am decreasing both the severity and the priority as this is just a tiny 
visual glitch.


[1] @dfaure yeah Dolphin uses that there ;)
Though imo dolphin should use iconName instead --> especially because of 
overlays, yet that is a different story.

Comment 3 Jeroen van Meeuwen (Kolab Systems) 2012-08-24 16:19:57 UTC

Resetting assignee to default as per bug #305719

Comment 4 Frank Reininghaus 2013-07-17 14:24:27 UTC

Reassignin to KIO because it seems to be an issue in KFileItem::pixmap(), as Matthias said (BTW, I can reproduce this in KDE 4.10.5).

Comment 5 Nate Graham 2018-04-25 22:01:53 UTC

Works for me in KDE Frameworks 5.45.