Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

readuce svg file size #1220

Merged
merged 1 commit into from
Apr 28, 2017
Merged

readuce svg file size #1220

merged 1 commit into from
Apr 28, 2017

Conversation

pingan1927
Copy link
Contributor

@pingan1927 pingan1927 commented Apr 27, 2017

fix issue #1176

First of all, thanks for your contribution! :-)

Please makes sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow antd's code convention.
  • Run npm run lint and fix those errors before submitting in order to keep consistent code style.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

@mention-bot
Copy link

@pingan1927, thanks for your PR! By analyzing the history of the files in this pull request, we identified @warmhug to be a potential reviewer.

@codecov
Copy link

codecov bot commented Apr 27, 2017

Codecov Report

Merging #1220 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1220      +/-   ##
==========================================
+ Coverage   69.76%   69.77%   +<.01%     
==========================================
  Files         218      218              
  Lines        4131     4129       -2     
  Branches     1218     1217       -1     
==========================================
- Hits         2882     2881       -1     
+ Misses       1248     1247       -1     
  Partials        1        1
Flag Coverage Δ
#rn 72.02% <ø> (ø) ⬆️
#web 67.43% <ø> (+0.01%) ⬆️
Impacted Files Coverage Δ
components/search-bar/index.web.tsx 54.73% <0%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07649b4...66d15a9. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 27, 2017

Codecov Report

Merging #1220 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1220      +/-   ##
==========================================
+ Coverage   69.76%   69.77%   +<.01%     
==========================================
  Files         218      218              
  Lines        4131     4129       -2     
  Branches     1218     1217       -1     
==========================================
- Hits         2882     2881       -1     
+ Misses       1248     1247       -1     
  Partials        1        1
Flag Coverage Δ
#rn 72.02% <ø> (ø) ⬆️
#web 67.43% <ø> (+0.01%) ⬆️
Impacted Files Coverage Δ
components/search-bar/index.web.tsx 54.73% <0%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07649b4...66d15a9. Read the comment docs.

@paranoidjk
Copy link
Contributor

不只icon下面有,看下svg.config.j s

@paranoidjk
Copy link
Contributor

这个先合掉了,其他的我晚上来改掉

@paranoidjk paranoidjk merged commit 3996e2b into master Apr 28, 2017
@paranoidjk paranoidjk deleted the optimize-icon branch April 28, 2017 07:43
lixiaoyang1992 pushed a commit to lixiaoyang1992/ant-design-mobile that referenced this pull request Apr 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants