Remove outdated DOM utilities and simplify marker SVG #11321
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DOM.setTransform(el, ...)
in favor ofel.style.transform = ...
.DOM.remove
in favor ofel.remove()
(supported everywhere except IE11).DOM.addEventListener
/DOM.removeEventListener
in favor ofel.addEventListener
(options / passive listeners supported everywhere except IE11).DOM.createSVG
utility in place ofDOM.createNS
which we only used for SVG.Marker before:
Marker after:
Launch Checklist
mapbox-gl-js
changelog:<changelog>Improved default marker performance.</changelog>