ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.


  Making monochrome breeze icons is tricky; you'll want to read 
https://community.kde.org/Guidelines_and_HOWTOs/Icon_Workflow_Tips if you 
haven't already.
  
  Also, screenshots using the Cuttlefish app are appreciated. :)

INLINE COMMENTS

> media-playlist-repeat.svg:7
> +     </defs>
> +     <path class="ColorScheme-Text" d="m16 5v2h-10c-1.662 0-3 1.338-3 
> 3v1h1v-1c0-1.108 0.892-2 2-2h10v2l3-2.5zm2 6v1c0 1.108-0.892 2-2 2h-10v-2l-3 
> 2.5 3 2.5v-2h10c1.662 0 3-1.338 3-3v-1z" color="#eff0f1" fill="currentColor"/>
>  </svg>

Remove `color="#eff0f1"`

> media-playlist-repeat.svg:7
> +     </defs>
> +     <path class="ColorScheme-Text" d="m16 5v2h-10c-1.662 0-3 1.338-3 
> 3v1h1v-1c0-1.108 0.892-2 2-2h10v2l3-2.5zm2 6v1c0 1.108-0.892 2-2 2h-10v-2l-3 
> 2.5 3 2.5v-2h10c1.662 0 3-1.338 3-3v-1z" color="#232629" fill="currentColor"/>
>  </svg>

remove `color="#232629"`

> media-repeat-none.svg:11
> +     <path class="ColorScheme-Text" d="m16 5v2h-10c-1.662 0-3 1.338-3 
> 3v1h1v-1c0-1.108 0.892-2 2-2h10v2l3-2.5zm-3 9h-7v-2l-3 2.5 3 2.5v-2h7z" 
> fill="currentColor"/>
> +     <path class="ColorScheme-NegativeText" d="m14.834 12-0.83398 0.83398 
> 1.666 1.666-1.666 1.666 0.83398 0.83398 1.666-1.666 1.666 1.666 
> 0.83398-0.83398-1.666-1.666 1.666-1.666-0.83398-0.83398-1.666 1.666z" 
> color="#da4453" fill="currentColor"/>
> +</svg>

remove `color="#da4453"`

REPOSITORY
  R266 Breeze Icons

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

To: astippich, #vdg, ngraham, ndavis
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to