fix(MultiSelect): get options label from an internal array instead of DOM to avoid inconsistency when the option contains text that resembles an HTML tag, ex. &gt;, &lt;

mrholek · mrholek · commit ffe1810e8463 · 2024-10-03T14:52:22.000+02:00
diff --git a/js/src/multi-select.js b/js/src/multi-select.js
@@ -649,7 +649,7 @@ class MultiSelect extends BaseComponent {
     }
 
     const value = String(element.dataset.value)
-    const text = element.textContent
+    const { text } = this._options.find(option => option.value === value)
 
     if (this._config.multiple && element.classList.contains(CLASS_NAME_SELECTED)) {
       this._deselectOption(value)

Original file line number	Diff line number	Diff line change
`@@ -649,7 +649,7 @@ class MultiSelect extends BaseComponent {`
`649`	`649`	`}`
`650`	`650`
`651`	`651`	`const value = String(element.dataset.value)`
`652`		`- const text = element.textContent`
	`652`	`+ const { text } = this._options.find(option => option.value === value)`
`653`	`653`
`654`	`654`	`if (this._config.multiple && element.classList.contains(CLASS_NAME_SELECTED)) {`
`655`	`655`	`this._deselectOption(value)`