Skip to content

Commit

Permalink
fix(ui5-textarea): remove the cloning of the value state message
Browse files Browse the repository at this point in the history
related to: #9347
  • Loading branch information
ndeshev committed Jul 5, 2024
1 parent 7901f7a commit ed640a4
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 10 deletions.
8 changes: 2 additions & 6 deletions packages/main/src/TextArea.ts
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ class TextArea extends UI5Element implements IFormInputElement {

_onfocusout(e: FocusEvent) {
const eTarget = e.relatedTarget as HTMLElement;
const focusedOutToValueStateMessage = eTarget?.shadowRoot?.querySelector(".ui5-valuestatemessage-root");
const focusedOutToValueStateMessage = eTarget && this.contains(eTarget);

this.focused = false;

Expand Down Expand Up @@ -577,7 +577,7 @@ class TextArea extends UI5Element implements IFormInputElement {
}

if (this.hasCustomValueState) {
return `${this.valueStateTypeMappings[this.valueState]}`.concat(" ", this.valueStateMessageText.map(el => el.textContent).join(" "));
return `${this.valueStateTypeMappings[this.valueState]}`.concat(" ", this.valueStateMessage.map(el => el.textContent).join(" "));
}

return `${this.valueStateTypeMappings[this.valueState]} ${this.valueStateDefaultText}`;
Expand Down Expand Up @@ -611,10 +611,6 @@ class TextArea extends UI5Element implements IFormInputElement {
return this.valueState === ValueState.Negative || this.valueState === ValueState.Critical || this.valueState === ValueState.Information;
}

get valueStateMessageText() {
return this.valueStateMessage.map(x => x.cloneNode(true));
}

get _valueStatePopoverHorizontalAlign(): `${PopoverHorizontalAlign}` {
return this.effectiveDir !== "rtl" ? "Start" : "End";
}
Expand Down
4 changes: 1 addition & 3 deletions packages/main/src/TextAreaPopover.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,7 @@

{{#*inline "valueStateMessage"}}
{{#if hasCustomValueState}}
{{#each valueStateMessageText}}
{{this}}
{{/each}}
<slot name="valueStateMessage"></slot>
{{else}}
{{valueStateDefaultText}}
{{/if}}
Expand Down
6 changes: 5 additions & 1 deletion packages/main/test/pages/TextArea.html
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,11 @@

<br>
<br>
<ui5-textarea rows="1" id="basic-textarea-error" placeholder="Placeholder" value-state="Negative"></ui5-textarea>
<ui5-textarea rows="1" id="basic-textarea-error" placeholder="Placeholder" value-state="Negative">
<div slot="valueStateMessage">
This is a random value state message text with a <a href="#" _target="blank">Link</a>.
</div>
</ui5-textarea>

<br>
<br>
Expand Down

0 comments on commit ed640a4

Please sign in to comment.