Re: [FlexJS] Use of innerHTML

2017-06-27 Thread Alex Harui
+1 for changing to textContent. We started out wanting to support IE8 which didn't have textContent, but IE8 is no longer our baseline. Thanks, -Alex On 6/27/17, 9:58 AM, "Harbs" wrote: >There’s a number of components which use innerHTML for setting text >values. (ImageAndTextButton, Label, St

[FlexJS] Use of innerHTML

2017-06-27 Thread Harbs
There’s a number of components which use innerHTML for setting text values. (ImageAndTextButton, Label, StringItemRenderer, TextButton) I just came across an article which advises against doing so.[1] Does anyone have a reason to not use append[2] and/or Node.textContent[3] and appendChild[4]?