-
Notifications
You must be signed in to change notification settings - Fork 261
Change from innerHTML to innerText #2330
Copy link
Copy link
Labels
area-file-formatsRelated to supported file formatsRelated to supported file formatsplatform-javascriptRelated to the JavaScript version of alphaTabRelated to the JavaScript version of alphaTabstate-acceptedThis is a valid topic to work on.This is a valid topic to work on.
Milestone
Metadata
Metadata
Assignees
Labels
area-file-formatsRelated to supported file formatsRelated to supported file formatsplatform-javascriptRelated to the JavaScript version of alphaTabRelated to the JavaScript version of alphaTabstate-acceptedThis is a valid topic to work on.This is a valid topic to work on.
Type
Fields
Give feedbackNo fields configured for Task.
Projects
Status
Done
Discussed in #2316
We currently use
innerHTMLto load various bits like alphaTex from the DOM. UsinginnerTextis what we actually need in most cases. Especially the case mentioned in the discussion above.Before actually changing we should check if there are any negative implications of changing. Especially in case of DOM scaping and how we parse things.