Skip to content

Conversation

@pixelipo
Copy link
Contributor

I've screwed up #7209 with rebase so, here it is again...

I've discovered a whole set of icons in settings app that needed to be fixed (mostly I just replaced them with already optimized icons found in core/img/* folders).

I've also fixed social icons as well as logo and favicon SVGs. Those last ones are quite critical, so test those carefully.

@nextcloud/designers

(finally) Fixes #4076

Signed-off-by: Marin Treselj <marin@pixelipo.com>
Signed-off-by: Marin Treselj <marin@pixelipo.com>
@codecov
Copy link

codecov bot commented Nov 20, 2017

Codecov Report

Merging #7231 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #7231   +/-   ##
=========================================
  Coverage     50.84%   50.84%           
  Complexity    24547    24547           
=========================================
  Files          1585     1585           
  Lines         93801    93801           
  Branches       1354     1354           
=========================================
  Hits          47697    47697           
  Misses        46104    46104

@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Nov 20, 2017
@MorrisJobke MorrisJobke added this to the Nextcloud 13 milestone Nov 20, 2017
@MorrisJobke MorrisJobke merged commit c277800 into master Nov 20, 2017
@MorrisJobke MorrisJobke deleted the fix-broken-svg-5 branch November 20, 2017 17:09
@skjnldsv
Copy link
Member

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lots of SVG source files broken since svgo optimization

4 participants