-
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:popover): add nzPopoverOverlayClickable
to disable click on mask
#8878
Conversation
This preview will be available after the AzureCI is passed. |
d55ccf5
to
372756c
Compare
372756c
to
90214cb
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8878 +/- ##
=======================================
Coverage 91.90% 91.91%
=======================================
Files 555 555
Lines 19711 19717 +6
Branches 2944 2946 +2
=======================================
+ Hits 18116 18122 +6
Misses 1268 1268
Partials 327 327 ☔ View full report in Codecov by Sentry. |
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.
1 comments, and please add unit tests
5774cbc
to
21cd1c9
Compare
a1e5e21
to
0ca53d8
Compare
0ca53d8
to
c07252b
Compare
bfdd018
to
a1a6c2a
Compare
@@ -141,6 +164,10 @@ export class NzPopoverTestComponent { | |||
@ViewChild('changePopover', { static: true, read: NzPopoverDirective }) | |||
changePopoverNzPopoverDirective!: NzPopoverDirective; | |||
|
|||
@ViewChild('hideTemplate', { static: false }) hideTemplate!: ElementRef; |
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.
这个变量好像没用?
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.
补充的测试
a1a6c2a
to
bf6f21c
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
nzPopoverOverlayClickable
to disable click on mask
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