-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(module:description): fix description not accept TemplateRef #5139
Conversation
This preview will be available after the AzureCI is passed. |
Clearly I didn't implement that in previous versions... |
Codecov Report
@@ Coverage Diff @@
## master #5139 +/- ##
=======================================
Coverage 91.46% 91.47%
=======================================
Files 547 547
Lines 13747 13768 +21
Branches 2130 2137 +7
=======================================
+ Hits 12574 12594 +20
+ Misses 708 707 -1
- Partials 465 467 +2
Continue to review full report at Codecov.
|
@@ -107,7 +117,9 @@ const defaultColumnMap: { [key in NzBreakpointEnum]: number } = { | |||
<tr class="ant-descriptions-row"> | |||
<ng-container *ngFor="let item of row; let isLast = last"> | |||
<td class="ant-descriptions-item-label" [colSpan]="item.span"> | |||
{{ item.title }} | |||
<ng-container *nzStringTemplateOutlet="item.title"> |
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 type definition of nzTitle also need to update
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.
lgtm
…ORRO#5139) * fix(module:description): fix description not accept TemplateRef close NG-ZORRO#5127 * fix: fix nzTitle type
close #5127
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