search

highcharts / highcharts-editor

https://www.highcharts.com/products/highcharts-editor
Other
249 stars 96 forks source link

tabletIcon is not defined #202

Open slavede opened 4 years ago

slavede commented 4 years ago

Expected behaviour

Syntac to be correct

Actual behaviour

When transpiled, minified and uglified code, highcharts-editor.js throws and error

ReferenceError: tabletIcon is not defined

which makes perfect sense when you look at the source of src/core-ui/highed.customizepage.js

151 | phoneIcon = highed.dom.cr('span', '', ''); 152 | tabletIcon = highed.dom.cr('span', '', ''), 153 | tabletIcon = highed.dom.cr('span', '', ''),

Notice ';' after phoneIcon. It should be comma.

Reproduction steps

Editor version

0.3.0

Affected browser(s)