Skip to content

Conversation

@pixelipo
Copy link
Contributor

@pixelipo pixelipo commented Jun 8, 2017

This one weight half the size of the old icon, while retaining much more readable (and editable) code. It is resized to 16px*16px because that is Nextcloud default.

It fills a little bit more space, since I noticed that the old icon looked smaller compared to the rest in the sidebar.

Code cleaned using Scour.

Signed-off-by: Marin Treselj marin@pixelipo.com

Signed-off-by: Marin Treselj <marin@pixelipo.com>
@pixelipo pixelipo requested a review from juliusknorr June 8, 2017 20:56
@pixelipo pixelipo added this to the 0.2.0 milestone Jun 8, 2017
@pixelipo pixelipo mentioned this pull request Jun 8, 2017
3 tasks
@codecov
Copy link

codecov bot commented Jun 8, 2017

Codecov Report

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

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   76.12%   76.12%           
=======================================
  Files          33       33           
  Lines         955      955           
=======================================
  Hits          727      727           
  Misses        228      228

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants