Bug 361192

Summary:	File managers use wrong breeze icons size.
Product:	[Plasma] Breeze	Reporter:	Yamashita Ren <lemaitre.dulotus>
Component:	Icons	Assignee:	visual-bugs-null
Status:	RESOLVED UPSTREAM
Severity:	normal	CC:	kainz.a, lemaitre.dulotus, tsujan2000
Priority:	NOR
Version First Reported In:	5.6.1
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Yamashita Ren 2016-03-30 16:02:09 UTC

Depending on the File Manager and the file type,  22x icons are used.
In Dolphin, they are used for everything but "special dirs" like Downloads, Music, Desktop...
In PCManFM-Qt, they are used for everything but directories.

Reproducible: Always

Steps to Reproduce:
1. Use Breeze icon theme.
2. Open Dolphin or PCManFM-QT.
3. Configure the icon size to something higher than 22, like 48 or 64.


Actual Results:  
Look at theses ugly 22x icons.

Expected Results:  
I should look at beautiful 48x icons.

Comment 1 Tsu Jan 2016-05-07 16:14:19 UTC

This is a Qt bug (in qiconloader.cpp -> directorySizeDistance()).

Comment 2 Tsu Jan 2016-05-07 16:16:52 UTC

Wrong address! Sorry, the bug is in qiconloader.cpp -> directoryMatchesSize().

Comment 3 andreas 2016-05-16 13:47:04 UTC

sorry but breeze-icons is also the wrong area can you add an screenshot?

Comment 4 Tsu Jan 2016-05-20 14:02:32 UTC

You could close this report. See https://codereview.qt-project.org/#/c/159800/

Comment 5 Yamashita Ren 2016-05-20 14:03:49 UTC

Thanks.