-
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(module:icon): support rotate #2891
Conversation
Deploy preview for ng-zorro-master ready! Built with commit c56850d |
Codecov Report
@@ Coverage Diff @@
## master #2891 +/- ##
==========================================
- Coverage 97.38% 97.36% -0.02%
==========================================
Files 526 526
Lines 10964 11033 +69
Branches 787 786 -1
==========================================
+ Hits 10677 10742 +65
- Misses 184 190 +6
+ Partials 103 101 -2
Continue to review full report at Codecov.
|
@@ -29,7 +29,9 @@ We are still adding two-tone icons right now, syncing to [antd](https://ant.desi | |||
| `[spin]` | Rotate icon with animation | `boolean` | `false` | | |||
| `[twoToneColor]` | Only support the two-tone icon. Specific the primary color. | `string (hex color)` | - | | |||
| `[iconfont]` | Type of the icon from iconfont | `string` | - | | |||
| `[nzRotate]` | Rotate degrees (since 7.0.0) | `number` | - | |
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.
en and zh doc are not the same
components/icon/demo/basic.ts
Outdated
@@ -8,6 +8,7 @@ import { Component } from '@angular/core'; | |||
<i nz-icon [type]="'setting'" [theme]="'fill'"></i> | |||
<i nz-icon [type]="'smile'" [theme]="'outline'"></i> | |||
<i nz-icon [type]="'sync'" [spin]="true"></i> | |||
<i nz-icon [type]="'smile'" [theme]="'outline'" [nzRotate]="180"></i> |
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.
should be [nzType]?
@vthinkxie I've updated the API in docs, demos, and tests. Please review. |
* feat(module:icon): support rotate close NG-ZORRO#2888 * fix: switch to API with prefix * docs: raname API in demo
* feat(module:icon): support rotate close NG-ZORRO#2888 * fix: switch to API with prefix * docs: raname API in demo
close #2888
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: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information