Diff Detail
- Repository
- R266 Breeze Icons
- Branch
- index (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 22239 Build 22257: arc lint + arc unit
This icon is visually nice, but it doesn't look 100% Breeze-y to me. I would see what feedback @ndavis has for the icon.
@cblack is right, breeze icons tend to be sharper, but there's no rule requiring sharpness and I've accepted similar icons recently (maybe I should edit those a bit, but they're still good). However, I'm not sure how we'll differentiate this from Dolphin if we make the corners sharper. For now, remove the dark blue bit in the top right. Sorry I missed this.
icons/apps/48/index.svg | ||
---|---|---|
29 | The shadow gradient here got moved way out of place: That happens sometimes when you use SVG cleaner. | |
35 | This one can't be seen, so get rid of it. |
icons/apps/48/index.svg | ||
---|---|---|
35 | Just realized this comment could be easily mistaken for me saying to get rid of the shadow because of how the phabricator UI looks. I'm talking about a blue rectangle. |