refactor(MultiSelect): cupdate tag delete button

Author: mrholek

js/src/multi-select.js

@@ -511,7 +511,7 @@ class MultiSelect extends BaseComponent {
     tag.dataset.value = value
     tag.innerHTML = text
 
-    const closeBtn = document.createElement('span')
+    const closeBtn = document.createElement('button')
     closeBtn.classList.add(CLASS_NAME_TAG_DELETE, 'close')
     closeBtn.setAttribute('aria-label', 'Close')
     closeBtn.innerHTML = '<span aria-hidden="true">&times;</span>'
@@ -655,10 +655,6 @@ class MultiSelect extends BaseComponent {
       return
     }
 
-    if (this._selection.length === 0 && (this._config.selectionType === 'tags' || this._config.selectionType === 'text')) {
-      this._searchElement.removeAttribute('size')
-    }
-
     if (this._selection.length > 0 && !this._config.multiple) {
       this._searchElement.placeholder = this._selection[0].text
       this._selectionElement.style.display = 'none'

scss/forms/_form-multi-select.scss

@@ -211,8 +211,9 @@
 }
 
 .form-multi-select-tag-delete {
+  padding: 0;
   margin-left: 6px;
-  cursor: pointer;
+  border: 0;
 }
 
 .form-multi-select.show {