Skip to content

feat(ui5-color-palette-popover): add placement property#12993

Merged
NHristov-sap merged 3 commits intomainfrom
BL_colorpalettepopover_placement
Feb 4, 2026
Merged

feat(ui5-color-palette-popover): add placement property#12993
NHristov-sap merged 3 commits intomainfrom
BL_colorpalettepopover_placement

Conversation

@NHristov-sap
Copy link
Contributor

This PR introduces new property placement of the <ui5-color-palette-popover>, which accepts values from PopoverPlacement enum (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

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Jan 30, 2026

🧹 Preview deployment cleaned up: https://pr-12993--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 30, 2026 11:59 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 2, 2026 08:31 Inactive
Copy link
Contributor

@GDamyanov GDamyanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 3, 2026 14:02 Inactive
@NHristov-sap NHristov-sap merged commit a8581d9 into main Feb 4, 2026
14 checks passed
@NHristov-sap NHristov-sap deleted the BL_colorpalettepopover_placement branch February 4, 2026 21:53
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 4, 2026 21:53 Inactive
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.19.0-rc.3 🎉

The release is available on v2.19.0-rc.3

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants