[kicker/kickoff] Add icon to hide application action
ClosedPublic

Authored by nicolasfella on Sep 1 2019, 11:52 AM.

Details

Summary

The icon name seems a bit weird to me, but it's what Dolphin uses for a similar purpose

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nicolasfella created this revision.Sep 1 2019, 11:52 AM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 1 2019, 11:52 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
nicolasfella requested review of this revision.Sep 1 2019, 11:52 AM
ngraham accepted this revision.Sep 1 2019, 7:34 PM
This revision is now accepted and ready to land.Sep 1 2019, 7:34 PM
This revision was automatically updated to reflect the committed changes.
hein added a comment.Sep 4 2019, 12:38 AM

Would it be better to add a new icon name to the theme instead of starting to use semantically dubious icons?

These names have annoyed me for years. I'll fix that.

Oh lol I already did it! https://cgit.kde.org/breeze-icons.git/commit/?id=bd07228a126446b35207fce903dcd3549604d208

The better icon names are view-visible and view-hidden.