Add application/x-audacity-project icon
ClosedPublic

Authored by cblack on Tue, Dec 31, 11:43 AM.

Details

Summary

BUG: 415722

Test Plan
  • Takes the audacity app icon, simplifies it slightly and monochromizes it. Getting it pixel-aligned was quite hard and I didn't do a very good job at it I think :)






Will clean the SVG file up and put it in icons-dark when the style is agreed upon

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.
broulik created this revision.Tue, Dec 31, 11:43 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptTue, Dec 31, 11:43 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Tue, Dec 31, 11:43 AM
broulik edited the test plan for this revision. (Show Details)
broulik edited the test plan for this revision. (Show Details)
ndavis requested changes to this revision.EditedTue, Dec 31, 9:23 PM
ndavis added a subscriber: ndavis.

Just needs some optimization and copy the files to icons-dark, then it'll be good to go

This revision now requires changes to proceed.Tue, Dec 31, 9:23 PM
broulik updated this revision to Diff 72526.Wed, Jan 1, 12:51 PM
  • Optimize
  • Copy to breeze dark
ngraham accepted this revision.Wed, Jan 1, 6:27 PM
ndavis accepted this revision.Thu, Jan 2, 2:01 AM
This revision is now accepted and ready to land.Thu, Jan 2, 2:01 AM
cblack added a subscriber: cblack.Thu, Jan 2, 2:10 AM

The foreground's colour contrast against the base of the colour icon should be higher IMO. (I have to squint to make out the emblem, which kinda defeats the purpose of an icon: making a filetype easier to see.)

Plus, dark mimetype icons almost never use a darker symbol overlaid on them, always a lighter one.

And the colours used seems to not be related at all to Audacity's brand colour (it's blue, not purple).

I would recolour the icons to look something more like this:

broulik added a comment.EditedThu, Jan 2, 7:06 AM

The icon is the midi icon which uses those colors.
Feel free to commandeer this revision.

cblack commandeered this revision.Thu, Jan 2, 5:49 PM
cblack added a reviewer: broulik.
cblack removed a subscriber: cblack.
cblack added a subscriber: cblack.
cblack updated this revision to Diff 72632.Thu, Jan 2, 5:50 PM

Refine colour palette of icons

cblack updated this revision to Diff 72635.Thu, Jan 2, 6:00 PM

Optimise with Scour

ndavis accepted this revision.Fri, Jan 3, 1:32 AM
This revision was automatically updated to reflect the committed changes.