-
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
feat(module:page-header): add page-header component #2732
Conversation
Deploy preview for ng-zorro-master ready! Built with commit 79fdda7 |
Codecov Report
@@ Coverage Diff @@
## master #2732 +/- ##
==========================================
+ Coverage 97.34% 97.34% +<.01%
==========================================
Files 552 559 +7
Lines 11587 11635 +48
Branches 827 827
==========================================
+ Hits 11279 11326 +47
- Misses 195 196 +1
Partials 113 113
Continue to review full report at Codecov.
|
0e20950
to
a4fa706
Compare
### nz-page-header | ||
| 参数 | 说明 | 类型 | 默认值 | | ||
| --- | --- | --- | --- | | ||
| nzTitle | title 文字 | `string|TemplateRef<void>` | - | |
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.
不仅仅是 文字
, 因为还支持 TemplateRef.
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.
类似于 ReactNode
,TemplateRef
应该不需要做特殊说明
@hsuanxyz This is not blocked anymore since ant design had released this component. Please check: https://ant.design/components/page-header-cn/. |
I would like to see it in the new release :D |
a4fa706
to
b9bfa01
Compare
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
* feat(module:page-header): add page-header component * test(module:page-header): add test * style(module:page-header): fix tslint close NG-ZORRO#2710 * docs(module:page-header): update docs * chore: sync class name * test(module:page-header): fix test
* feat(module:page-header): add page-header component * test(module:page-header): add test * style(module:page-header): fix tslint close NG-ZORRO#2710 * docs(module:page-header): update docs * chore: sync class name * test(module:page-header): fix test
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: #2710
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Ref ant-design/ant-design#13637