Opened 8 years ago
Closed 8 years ago
#38556 closed enhancement (fixed)
Twenty Seventeen: supportsInlineSVG is unnecessarily called twice
Reported by: | tywayne | Owned by: | davidakennedy |
---|---|---|---|
Milestone: | 4.7 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description
In /assets/js/global.js
we set a class based on SVG support using supportsInlineSVG
like so:
supportsInlineSVG(); if ( true === supportsInlineSVG() ) { document.documentElement.className = document.documentElement.className.replace( /(\s*)no-svg(\s*)/, '$1svg$2' ); }
There is no need to call supportsInlineSVG
, and then immediately call it again in the logic check.
Attachments (1)
Change History (4)
Note: See
TracTickets for help on using
tickets.
In 39081: