[RFC] Fix Display Configuration icon margins
ClosedPublic

Authored by ngraham on Nov 14 2019, 4:00 AM.

Details

Summary

The icon's old margins were too large, causing it to stick out.

BUG: 400087
FIXED-IN: 5.66

Test Plan

Diff Detail

Repository
R242 Plasma Framework (Library)
Branch
fix-xrandr-icon (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 18776
Build 18794: arc lint + arc unit
ngraham created this revision.Nov 14 2019, 4:00 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptNov 14 2019, 4:00 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
ngraham requested review of this revision.Nov 14 2019, 4:00 AM

It's blurry in your screenshot because you're scaling a 22px icon up to 32px. The solution is to make a 32px version named 32-32-preferences-desktop-display-randr. BTW, the icon is not actually aligned with the grid. It's a little off here and there. You should be able to fix it easily with the node tool by snapping nodes to the grid.

ndavis added inline comments.Nov 15 2019, 9:00 AM
src/desktoptheme/breeze/icons/preferences.svg
4

Wrong default colors. Just replace the whole stylesheet with this one: https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Breeze

36

You have 2 embedded stylesheets, although I doubt the 2nd one actually works since the ID isn't correct. You can delete the 2nd one.

73

Hardcoded color

74

Hardcoded color. Inkscape does this to any element of the SVG you modify.

ngraham updated this revision to Diff 69815.EditedNov 15 2019, 3:57 PM
ngraham marked 4 inline comments as done.
  • Fix stylesheet and color-changing ability
  • Align to grid; vertical lines still blurry, not sure why

I figured it out. The 22px version needs the 22-22- prefix in its ID.

Hmm, if I change that, the icon disappears entirely when displayed at the 22px size:

Never mind, I wasn't deleting the cache files properly after rebuilding. When I do that, the monochrome icons don't get used at all and it reverts to the colorful one:

@ndavis or @trickyricky26, could I ping you on this?

ndavis added a comment.Dec 6 2019, 4:27 PM

Never mind, I wasn't deleting the cache files properly after rebuilding. When I do that, the monochrome icons don't get used at all and it reverts to the colorful one:

The color vs monochrome issue strikes again :(

Maybe remove the size prefix from the 32px version? If that doesn't work, I've got nothing.

ngraham updated this revision to Diff 71346.Dec 12 2019, 10:58 AM

Finally fix this thing

ngraham edited the summary of this revision. (Show Details)Dec 12 2019, 10:59 AM
ngraham edited the test plan for this revision. (Show Details)
ndavis accepted this revision.Dec 12 2019, 4:49 PM
This revision is now accepted and ready to land.Dec 12 2019, 4:49 PM
This revision was automatically updated to reflect the committed changes.