-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat(schematics): add fix icon schematic #2238
Conversation
a41ecf1
to
71d6542
Compare
Codecov Report
@@ Coverage Diff @@
## master #2238 +/- ##
=========================================
Coverage ? 95.92%
=========================================
Files ? 485
Lines ? 12122
Branches ? 1621
=========================================
Hits ? 11628
Misses ? 139
Partials ? 355 Continue to review full report at Codecov.
|
@@ -27,7 +27,7 @@ timeline: true | |||
* 支持多色图标。 | |||
* 对于内建图标的更换可以提供更多 API,而不需要进行样式覆盖。 | |||
|
|||
我们尽可能地在不增加包体积的前提下对旧的 API 进行了兼容,你无需修改代码,但可能需要进行一些配置。关于 icon 的更多信息,请阅读 Icon API 的[“Svg 图标”](/components/icon/zh#svg-图标)和[“静态加载与动态加载”两节](/components/icon/zh#静态加载与动态加载)。 | |||
我们尽可能地在不增加包体积的前提下对旧的 API 进行了兼容,你无需修改代码,但可能需要进行一些配置。关于 icon 的更多信息,请阅读 Icon API 的[“Svg 图标”](/components/icon/zh#svg-图标)和[“静态加载与动态加载”](/components/icon/zh#静态加载与动态加载)两节。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could use URI encode here too.
Do we need to run this when user runs |
Will automatic run |
components/icon/doc/index.en-US.md
Outdated
@@ -7,6 +7,11 @@ hasPageDemo: true | |||
|
|||
Semantic vector graphics. | |||
|
|||
<blockquote style="border-color: orange;"> | |||
<p><strong>If the icon cannot be displayed or the console has an error, Please run the following command to fix it <code>ng g ng-zorro-antd:fix-icon</code>.</strong></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The p in Please run should be lower case.
71d6542
to
aee32d6
Compare
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #2171
What is the new behavior?
Does this PR introduce a breaking change?
Other information