-
Notifications
You must be signed in to change notification settings - Fork 238
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): feat add customClass for customer #129
Conversation
Codecov Report
@@ Coverage Diff @@
## master #129 +/- ##
==========================================
+ Coverage 86.51% 86.52% +0.01%
==========================================
Files 221 221
Lines 6983 6985 +2
Branches 978 978
==========================================
+ Hits 6041 6044 +3
Misses 550 550
+ Partials 392 391 -1
Continue to review full report at Codecov.
|
className maybe better than customClass? |
@@ -20,6 +20,7 @@ Properties | Descrition | Type | Default | |||
| mask | Whether to display the mask background layer | Boolean | false | | |||
| overlay | Popup layer content | TemplateRef | - | | |||
| onSelect | when an option is selected | (node: any, index?: number): void | - | | |||
| className | Custom popover class | string | am-popover | |
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.
| className | Custom popover class, e.g. "am-popover-${your custom class}" | string | am-popover |
much better?
* feat(module: popover): feat add customCls for customer * feat(module: popover): feat modify className for customer * docs(module:popover): update docs className descrpition (#120)
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