search

superman66 / vue-highcharts

The Component of Vue 2.x for highcharts
https://codesandbox.io/s/jjyqvv0k13
MIT License
115 stars 48 forks source link

Align with best practices for production component #33

Closed n-zeplo closed 6 years ago

n-zeplo commented 6 years ago

  1. Vue community has implemented lower case props as the standard for props with dashes. It is enforced by eslint vue plugin. So I changed Highchart to highchart for the props

  2. Remove sourcemaps for production builds to comply with CSP best practice of not using eval. Also, reduces size of included component in others codebase.

superman66 commented 6 years ago

Thanks~