Add flameshot tray icons
ClosedPublic

Authored by axionl on Sun, Mar 8, 4:48 AM.

Details

Summary

Add the system tray icon for flameshot. This patch adds some icons following the breeze style.

Test Plan

Before:

After:

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.
axionl created this revision.Sun, Mar 8, 4:48 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald TranscriptSun, Mar 8, 4:48 AM
axionl requested review of this revision.Sun, Mar 8, 4:48 AM
axionl retitled this revision from Add flameshot tray icons to [breeze-icons] Add flameshot tray icons.Sun, Mar 8, 4:57 AM
ngraham requested changes to this revision.Sun, Mar 8, 3:25 PM
ngraham added a subscriber: ngraham.

Thanks, this will be nice to have once the issue identified in my comment is resolved.

icons-dark/status/22/flameshot-tray.svg
3

Hardcoded colors alert! :)

You need to embed the stylesheet and make each path use it. Follow the pattern in other monochrome breeze icons, and see how to do that at https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#Embedding_stylesheets_in_SVGs

This revision now requires changes to proceed.Sun, Mar 8, 3:25 PM
axionl updated this revision to Diff 77223.Sun, Mar 8, 4:35 PM

Update flameshot tray icon.

Nice!

Do you think you could also make 16px versions?

axionl marked an inline comment as done.Sun, Mar 8, 5:06 PM

Nice!

Do you think you could also make 16px versions?

Sorry, maybe not. The 16px space is too small to get the correct angle to convert 1px thick strokes to paths on Inkscape.

ndavis added a subscriber: ndavis.Sun, Mar 8, 11:10 PM

It's a systray icon, so it doesn't need to have a 16px version

ndavis accepted this revision.Sun, Mar 8, 11:12 PM
ngraham accepted this revision.Mon, Mar 9, 5:32 PM

Derp, silly me.

This revision is now accepted and ready to land.Mon, Mar 9, 5:32 PM
ngraham retitled this revision from [breeze-icons] Add flameshot tray icons to Add flameshot tray icons.Mon, Mar 9, 5:33 PM
This revision was automatically updated to reflect the committed changes.