-
Notifications
You must be signed in to change notification settings - Fork 66
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: add PopoverVariant enum with the arrow variant #6499
Conversation
0bf7c6d
to
f38b1c0
Compare
|
||
private final String variant; | ||
|
||
PopoverVariant(String variant) { |
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.
Could we have a test?
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.
I think we don't have tests for other variant enums, not sure how to test this one.
ba093c1
to
a886332
Compare
...rent/vaadin-popover-flow/src/main/java/com/vaadin/flow/component/popover/PopoverVariant.java
Outdated
Show resolved
Hide resolved
...rent/vaadin-popover-flow/src/main/java/com/vaadin/flow/component/popover/PopoverVariant.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Sascha Ißbrücker <[email protected]>
Quality Gate passedIssues Measures |
This ticket/PR has been released with Vaadin 24.5.0.alpha9 and is also targeting the upcoming stable 24.5.0 version. |
Description
Based on vaadin/web-components#7550
Note: the
arrow
variant is available for both Lumo and Material.Type of change