D24598: Update elisa icon

This revision was automatically updated to reflect the committed changes. Closed by commit R266:a51651fd91a2: Update elisa icon (authored by astippich). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24598?vs=68192=68470 REVISION DETAIL

D24598: Update elisa icon

trickyricky26 accepted this revision. This revision is now accepted and ready to land. REPOSITORY R266 Breeze Icons BRANCH elisa_updated (branched from master) REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26,

D24598: Update elisa icon

ngraham accepted this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt, LeGast00n, carneirogustavo, jguidon,

D24598: Update elisa icon

astippich updated this revision to Diff 68192. astippich added a comment. - update and save as optimized svg REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24598?vs=67836=68192 BRANCH elisa_updated (branched from master) REVISION DETAIL

D24598: Update elisa icon

ngraham added a comment. Let's get this in! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt, LeGast00n,

D24598: Update elisa icon

ngraham accepted this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt, LeGast00n, carneirogustavo, jguidon,

D24598: Update elisa icon

mgallien accepted this revision. mgallien added a comment. Thanks REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien, trickyricky26 Cc: trickyricky26, GB_2, kde-frameworks-devel, stuartm, daerny, mfraser, mnesbitt,

D24598: Update elisa icon

ngraham added a comment. In D24598#548335 , @astippich wrote: > This is with the tape window moved up 1px > F7608368: elisaBigNew2.png > When I look at images of real cassettes, they are not centered,

D24598: Update elisa icon

astippich added a comment. This is with the tape window moved up 1px F7608368: elisaBigNew2.png When I look at images of real cassettes, they are not centered, hence why I only did this slight adjustment. If everyone is okay with this, I will

D24598: Update elisa icon

trickyricky26 requested changes to this revision. trickyricky26 added a comment. This revision now requires changes to proceed. Please optimize the SVG files using one of these methods: https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips#SVG_optimization In D24598#546420

D24598: Update elisa icon

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Lovely! REPOSITORY R266 Breeze Icons BRANCH elisa_updated (branched from master) REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien Cc:

D24598: Update elisa icon

astippich added a comment. F7585764: elisaSmallNew.png F7585766: elisaBigNew.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien Cc:

D24598: Update elisa icon

astippich updated this revision to Diff 67836. astippich added a comment. - update REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24598?vs=67799=67836 BRANCH elisa_updated (branched from master) REVISION DETAIL https://phabricator.kde.org/D24598

D24598: Update elisa icon

mgallien added a comment. In D24598#546135 , @astippich wrote: > I have no intention on taking over D12992 as I would like rather like to code for Elisa. I am no designer. > I was unsatisfied with the

D24598: Update elisa icon

ngraham added a comment. Can you make the gradient extend all the way to the top of the white center area? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D24598 To: astippich, #vdg, ngraham, mgallien Cc: GB_2, kde-frameworks-devel, stuartm, daerny, mfraser,

D24598: Update elisa icon

astippich added a comment. I have no intention on taking over D12992 as I would like rather like to code for Elisa. I am no designer. I was unsatisfied with the icon and @mgallien agreed on some slight adjustments to the current one. D12992

D24598: Update elisa icon

GB_2 added a comment. I don't think it makes sense to change the icon here if D12992 already does it. If you want it to progress then take over the revision and work on the completely new style there or make a ping comment and have some patience.

D24598: Update elisa icon

astippich added a comment. old icons 22px F7580424: elisaSmallOld.png 48px F7580438: elisaBigOld.png updated icons 22px F7580429: elisaSmall.png 48px

D24598: Update elisa icon

astippich created this revision. astippich added reviewers: VDG, ngraham, mgallien. astippich added a project: Elisa. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. astippich requested review of this revision. REVISION SUMMARY Since D12992