Skip to content

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

@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

);
}

describe("Color Popover Palette general interaction tests", () => {
Copy link
Contributor

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?

Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

add since

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 3, 2026 14:02 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants