From b48dcf5947021efee097bdb7fe61e0ba074d99df Mon Sep 17 00:00:00 2001 From: Lan Nguyen Thuy Date: Thu, 3 Jul 2025 13:07:26 +0700 Subject: [PATCH 1/3] remove select in option in uui-select --- packages/uui-select/lib/uui-select.element.ts | 24 +++---------------- packages/uui-select/lib/uui-select.story.ts | 13 +++++----- packages/uui-select/lib/uui-select.test.ts | 2 +- 3 files changed, 10 insertions(+), 29 deletions(-) diff --git a/packages/uui-select/lib/uui-select.element.ts b/packages/uui-select/lib/uui-select.element.ts index 41852be11..eef961fda 100644 --- a/packages/uui-select/lib/uui-select.element.ts +++ b/packages/uui-select/lib/uui-select.element.ts @@ -11,7 +11,6 @@ declare global { name: string; value: string; group?: string; - selected?: boolean; disabled?: boolean; } } @@ -87,7 +86,6 @@ export class UUISelectElement extends UUIFormControlMixin(LitElement, '') { name: string; value: string; group?: string; - selected?: boolean; disabled?: boolean; }` */ @@ -173,8 +171,6 @@ export class UUISelectElement extends UUIFormControlMixin(LitElement, '') { if (changedProperties.has('options')) { this._extractGroups(); this._values = this.options.map(option => option.value); - const selected = this.options.find(option => option.selected); - this.value = selected ? selected.value : ''; } if (changedProperties.has('value')) { @@ -200,13 +196,9 @@ export class UUISelectElement extends UUIFormControlMixin(LitElement, '') { private _renderOption( name: string, value: string, - selected: boolean | undefined, disabled: boolean | undefined, ) { - return html``; } @@ -224,12 +216,7 @@ export class UUISelectElement extends UUIFormControlMixin(LitElement, '') { )}> ${this.options.map(option => option.group === group - ? this._renderOption( - option.name, - option.value, - option.selected, - option.disabled, - ) + ? this._renderOption(option.name, option.value, option.disabled) : '', )} `, @@ -259,12 +246,7 @@ export class UUISelectElement extends UUIFormControlMixin(LitElement, '') { ${this.options .filter(option => !option.group) .map(option => - this._renderOption( - option.name, - option.value, - option.selected, - option.disabled, - ), + this._renderOption(option.name, option.value, option.disabled), )} `; } diff --git a/packages/uui-select/lib/uui-select.story.ts b/packages/uui-select/lib/uui-select.story.ts index b11dfa2c7..e8b400282 100644 --- a/packages/uui-select/lib/uui-select.story.ts +++ b/packages/uui-select/lib/uui-select.story.ts @@ -52,10 +52,7 @@ const options: Array