ndavis added a comment.

  @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.

INLINE COMMENTS

> index.svg:29
> +  </linearGradient>
> +  <linearGradient id="linearGradient69417" x1="39.355347" x2="41.273647" 
> y1="10.5268" y2="15" gradientTransform="translate(-391.5697,-499.8)" 
> gradientUnits="userSpaceOnUse">
> +   <stop stop-color="#292c2f" offset="0"/>

The shadow gradient here got moved way out of place: F8257499: 
Screenshot_20200424_074454.png <https://phabricator.kde.org/F8257499>

That happens sometimes when you use SVG cleaner.

> index.svg:35
> + <g>
> +  <rect x="4" y="4" width="40" height="39" rx="6" ry="6" fill="#147eb8"/>
> +  <rect x="4" y="4" width="40" height="40" rx="6" ry="6" fill="#147eb8"/>

This one can't be seen, so get rid of it.

REPOSITORY
  R266 Breeze Icons

BRANCH
  index (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27252

To: mbruchert, camiloh, #vdg, ndavis
Cc: cblack, ndavis, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

Reply via email to