Use breeze icons for categories
ClosedPublic

Authored by guoyunhe on Mar 7 2020, 3:43 PM.

Details

Summary

Before:


After:

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 23405
Build 23423: arc lint + arc unit
guoyunhe created this revision.Mar 7 2020, 3:43 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 7 2020, 3:43 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
guoyunhe requested review of this revision.Mar 7 2020, 3:43 PM
guoyunhe edited the summary of this revision. (Show Details)Mar 7 2020, 3:47 PM
guoyunhe added a reviewer: Plasma.
guoyunhe updated this revision to Diff 77174.Mar 7 2020, 3:54 PM

Change object-group to object

davidre added a subscriber: davidre.Mar 7 2020, 4:10 PM

There is something wrong on your system, this is how before looks for me:

There is something wrong on your system, this is how before looks for me:

Please ignore the content. I only want to compare the left panel.

There is something wrong on your system, this is how before looks for me:

Please ignore the content. I only want to compare the left panel.

Yes there is still a difference in the left panel in my screenshot compared to your before screenshot. Not pixelated and all categories colorful

guoyunhe edited the summary of this revision. (Show Details)Mar 7 2020, 4:13 PM

There is something wrong on your system, this is how before looks for me:

Please ignore the content. I only want to compare the left panel.

Yes there is still a difference in the left panel in my screenshot compared to your before screenshot. Not pixelated and all categories colorful

The "pixelated" icon only appears to HiDPI display. The "monochrome" icon only appears when the system has dejavu-fonts installed. It looks different in different distros.

ngraham added a reviewer: apol.Mar 8 2020, 3:48 PM
ngraham added a subscriber: ngraham.

Hmm, your before and after screenshots look identical to me.

What's the reason for this? To make them look better in High DPI mode? I thought that was already fixed with D27515: Icon: fix rendering of image: urls on High DPI

apol accepted this revision.Mar 10 2020, 12:15 AM

Don't land before D27913.

This revision is now accepted and ready to land.Mar 10 2020, 12:15 AM
Closed by commit R119:7a6e8f3bc52a: Use breeze icons for categories (authored by Guo Yunhe <i@guoyunhe.me>). · Explain WhyMar 11 2020, 8:36 AM
This revision was automatically updated to reflect the committed changes.