Add new baloo icons
ClosedPublic

Authored by astippich on Oct 26 2019, 8:22 AM.

Details

Reviewers
ngraham
bruns
Group Reviewers
VDG
Maniphest Tasks
T7878: Create or find Icon
Commits
R266:61331d6ce20a: Add new baloo icons
Summary

Add new baloo 48px icon and update the preference-desktop-baloo
icon to the same style.

Closes T7878

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.
astippich created this revision.Oct 26 2019, 8:22 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 26 2019, 8:22 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
astippich requested review of this revision.Oct 26 2019, 8:22 AM

Before:


which is basically a copy of the kfind icon

ngraham edited the summary of this revision. (Show Details)Oct 26 2019, 5:42 PM
ngraham accepted this revision.Oct 26 2019, 5:45 PM

Lovely.

This revision is now accepted and ready to land.Oct 26 2019, 5:45 PM

Some of my thoughts on the new preferences-desktop-search icons (D24959#554523) also apply here: I think that lines of the magnifying glass symbol feel a bit too thick for Breeze, and IMO the large glass-to-handle ratio doesn't feel quite right, although I realize you have to fit that paw symbol in there. Also, this kind of icon is a classic candidate for the 45° degree shadow.

I would also try making the paw symbol a bit more simple, perhaps by removing the ellipses between the bottom and the claws themselves, in order to make the design less visually noisy. Then it might also be possible to make the paw smaller in the 32px version.

Maybe we can also come up with a more interesting background than a blue circle, which is quite bland, considering this is an application icon.

astippich updated this revision to Diff 69501.Nov 9 2019, 11:21 AM
  • tweak icon and shadows
ngraham accepted this revision.Wed, Nov 13, 9:33 PM

the scalable test fails with this one. Is the right solution to just delete the 32px symlink under .../preferences?

Yes, that's correct.

astippich updated this revision to Diff 69857.Sat, Nov 16, 7:32 PM
  • tweak icon and shadows
  • remove baloo symlink
This revision was automatically updated to reflect the committed changes.