-
Notifications
You must be signed in to change notification settings - Fork 279
feat(ui5-color-palette-popover): add placement property #12993
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
base: main
Are you sure you want to change the base?
Conversation
|
🚀 Deployed on https://pr-12993--ui5-webcomponents-preview.netlify.app |
GDamyanov
left a comment
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
| ); | ||
| } | ||
|
|
||
| describe("Color Popover Palette general interaction tests", () => { |
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.
Why do we change the formatting of all of these tests?
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 tests were not tabulated by TAB but by SPACEs, but you're right, it is matter of another PR, I'll publish just my changes here.
| /** | ||
| * Determines on which side the component is placed at. | ||
| * @default "Bottom" | ||
| * @public |
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.
add since
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.
added
This PR introduces new property
placementof the<ui5-color-palette-popover>, which accepts values fromPopoverPlacementenum (the same used by the<ui5-popover>and<ui5-responsive-popover>. Possible values are Top, Bottom, Start and End, and the default value of the property is Bottom.The value of this property is passed directly to the internal
<ui5-responsive-popover>in order to display the color palette popover at desired position based to the opener.JIRA: BGSOFUIBALKAN-9994