BVSelect-VanillaJS icon indicating copy to clipboard operation
BVSelect-VanillaJS copied to clipboard

Dynamicly creation option elements breaks when value contain spaces

Open AforDesign opened this issue 1 year ago • 0 comments

When option elements are created dynamicly, the values need to be encapsulated. Right now, this is not the case, resulting in broken html when values contain spaces.

To fix this, change:

                document.getElementById(this.selector).insertAdjacentHTML('beforeend',
                    "<option "+change_img+" "+change_icon+" "+change_separator+" "+change_disabled+" value="+properties.options[i].value+" >"+properties.options[i].inner_text+"</option>");

To:

                document.getElementById(this.selector).insertAdjacentHTML('beforeend',
                    "<option "+change_img+" "+change_icon+" "+change_separator+" "+change_disabled+" value=\""+properties.options[i].value+"\" >"+properties.options[i].inner_text+"</option>");

AforDesign avatar Dec 09 '22 01:12 AforDesign