D18533: Improve the Notfication Bell Icon by using the KAlarm design

This revision was automatically updated to reflect the committed changes. Closed by commit R266:19c936621729: Improve the Notfication Bell Icon by using the KAlarm design (authored by trickyricky26, committed by ngraham). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. I love it! REPOSITORY R266 Breeze Icons BRANCH improve-notification-bell-icon (branched from master) REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg,

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: abetts, ngraham, ndavis, djarvie, kde-frameworks-devel, michaelh, bruns

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 updated this revision to Diff 50703. trickyricky26 added a comment. - Use a less flat gradient with a silver ringer REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18533?vs=50280=50703 BRANCH improve-notification-bell-icon (branched

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ndavis added a comment. In D18533#403354 , @ngraham wrote: > In D18533#403351 , @ndavis wrote: > > > The ringer inside the bell has contrast issues with the standard Breeze Dark background color:

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ngraham added a comment. In D18533#403351 , @ndavis wrote: > The ringer inside the bell has contrast issues with the standard Breeze Dark background color: F6581679: Screenshot_20190201_131929.png

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ndavis added a comment. The ringer inside the bell has contrast issues with the standard Breeze Dark background color: F6581679: Screenshot_20190201_131929.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To:

D18533: Improve the Notfication Bell Icon by using the KAlarm design

abetts added a comment. In D18533#403337 , @ndavis wrote: > F6581666: Screenshot_20190201_131014.png > > The colors are not very good, but this is what I was talking about. It looks more 3D. I

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ndavis added a comment. F6581666: Screenshot_20190201_131014.png The colors are no so good, but this is what I was talking about. It looks more 3D. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To:

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ngraham added a comment. How about putting a line under the little nubbin on top? F6581659: Bell with nubbin.png REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: ngraham, ndavis,

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 added a comment. I changed the shape a bit, but making it less flat is difficult: Linear gradients look off because of the curvature of the bell. Other options are blurring a curved object or using mesh gradients, but I'm not sure how well, if it all, these will work in

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ngraham added a comment. Any update/progress on this? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: ngraham, ndavis, djarvie, kde-frameworks-devel, michaelh, bruns

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ngraham added a comment. I actually think this new icon is pretty great as-is (at the minimum, it's much better than the existing icon) but I agree that it could benefit from a bit more depth. Here's another idea: maybe tilt the bell a bit to make it looks like it's in the middle of

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 planned changes to this revision. trickyricky26 added a comment. I will try to visually improve the design. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: ndavis, djarvie, kde-frameworks-devel, michaelh, ngraham,

D18533: Improve the Notfication Bell Icon by using the KAlarm design

ndavis added a comment. In D18533#400011 , @trickyricky26 wrote: > I have made some slight adjustments from the original KAlarm design, such as a shadow line at the bottom instead of orange, to better fir our style. > Should I make those

D18533: Improve the Notfication Bell Icon by using the KAlarm design

cgiboudeaux added a subscriber: djarvie. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: djarvie, kde-frameworks-devel, michaelh, ngraham, bruns

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 edited the summary of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 added a comment. I have made some slight adjustments from the original KAlarm design, such as a shadow line at the bottom instead of orange, to better fir our style. Should I make those changes in the original icon as well? REPOSITORY R266 Breeze Icons REVISION DETAIL

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 edited the summary of this revision. trickyricky26 edited the test plan for this revision. trickyricky26 added a reviewer: VDG. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D18533 To: trickyricky26, #vdg Cc: kde-frameworks-devel, michaelh, ngraham,

D18533: Improve the Notfication Bell Icon by using the KAlarm design

trickyricky26 created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. trickyricky26 requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH improve-notification-bell-icon (branched from master) REVISION DETAIL