Add icons for Touch Screen and Screen Edge KCMs
ClosedPublic

Authored by ndavis on Jan 23 2019, 7:07 PM.

Details

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ndavis created this revision.Jan 23 2019, 7:07 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 23 2019, 7:07 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
ndavis requested review of this revision.Jan 23 2019, 7:07 PM
ndavis edited the test plan for this revision. (Show Details)Jan 23 2019, 7:10 PM

Maybe preferences-desktop-gestures-screenedges should have the cursor near to the hot corner to reinforce the concept? Otherwise it kind of feels like an icon for the cursors KCM.

Maybe preferences-desktop-gestures-screenedges should have the cursor near to the hot corner to reinforce the concept? Otherwise it kind of feels like an icon for the cursors KCM.

The issue with that is that it would be the only one with an uncentered symbol. I can make an icon for the Cursors KCM with no desktop background, so it shouldn't be that bad.

Cool. I think the issue here is that the hotcorner bit is simultaneously too small to be easily noticeable, but visible enough to subtly unbalance the icon. I might recommend experimenting with making the hotcorner highlight more visually prominent and then adding a screen edge highlight on the right side to balance it out.

Or here's an idea: do those things and then remove the cursor icon from it entirely.

ndavis updated this revision to Diff 50154.Jan 24 2019, 12:19 AM

Change style of screen edges icon

ndavis edited the test plan for this revision. (Show Details)Jan 24 2019, 12:20 AM
ngraham accepted this revision.Jan 24 2019, 12:27 AM

Love it. Shipit!

This revision is now accepted and ready to land.Jan 24 2019, 12:27 AM
ndavis updated this revision to Diff 50156.Jan 24 2019, 1:00 AM

Add change to breeze dark

This revision was automatically updated to reflect the committed changes.