D24957: Add new baloo icons

This revision was automatically updated to reflect the committed changes. Closed by commit R266:61331d6ce20a: Add new baloo icons (authored by astippich). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24957?vs=69857=69859 REVISION DETAIL

D24957: Add new baloo icons

astippich updated this revision to Diff 69857. astippich added a comment. - tweak icon and shadows - remove baloo symlink REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24957?vs=69501=69857 BRANCH newBalooIcon REVISION DETAIL

D24957: Add new baloo icons

ngraham added a comment. Yes, that's correct. REPOSITORY R266 Breeze Icons BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: trickyricky26, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24957: Add new baloo icons

astippich added a comment. the scalable test fails with this one. Is the right solution to just delete the 32px symlink under .../preferences? REPOSITORY R266 Breeze Icons BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc:

D24957: Add new baloo icons

ngraham accepted this revision. REPOSITORY R266 Breeze Icons BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: trickyricky26, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24957: Add new baloo icons

astippich updated this revision to Diff 69501. astippich added a comment. - tweak icon and shadows REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24957?vs=68777=69501 BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957

D24957: Add new baloo icons

astippich added a comment. F7686927: test32.png F7686929: test48.png REPOSITORY R266 Breeze Icons BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham,

D24957: Add new baloo icons

trickyricky26 added a comment. 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

D24957: Add new baloo icons

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Lovely. REPOSITORY R266 Breeze Icons BRANCH newBalooIcon REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: kde-frameworks-devel,

D24957: Add new baloo icons

ngraham edited the summary of this revision. ngraham added a task: T7878: Create or find Icon. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24957: Add new baloo icons

bruns added a comment. LGTM REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D24957: Add new baloo icons

astippich added a comment. Before: F7673421: test.png which is basically a copy of the kfind icon REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc: kde-frameworks-devel,

D24957: Add new baloo icons

astippich added a comment. F7673101: baloo48.png F7673103: baloo32.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24957 To: astippich, #vdg, ngraham, bruns Cc:

D24957: Add new baloo icons

astippich created this revision. astippich added reviewers: VDG, ngraham, bruns. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. astippich requested review of this revision. REVISION SUMMARY Add new baloo 48px icon and update the preference-desktop-baloo