search

highcharts / highcharts-vue

Other
686 stars 150 forks source link

Suggestion: HighchartsVue.use(feature) for easier registration of features #138

Closed richardeschloss closed 1 year ago

richardeschloss commented 4 years ago

Hi, I already submitted the PR #137 which helps describes the suggestion and provides the suggested solution. I hope you like it. I find it allows for less code and documentation.

In a nutshell, the idea is:

import HighchartsVue from 'highcharts-vue'
HighchartsVue.use('stockChart') // Registers the stockChart feature.
Denyllon commented 4 years ago

@richardeschloss

Thank you for contribution to development process of our package! First of all, wanted to apologize for a bit of late in replying you. I will review your PR this week, and let you know in case of some doubts.

Kind regards!

richardeschloss commented 4 years ago

Ok cool. I'm here to be helpful anyway I can. I'm still toying with ideas on my end, and if I can help squash other issues I'll let you know. This project's success is also my success, because I use highcharts quite a bit.

TheSeg commented 4 years ago

Any progress on rolling this PR out?

Denyllon commented 4 years ago

Hi @TheSeg ,

Unfortunately we did not take a closer look on it, because encountered some problem with that PR, and needed to investigate it a bit deeper. Meanwhile, the Vue v3 has came out and we've considered to re-built this wrapper so that it will be able to work with the latest Vue release, so most probably we will face all of the issues while implementing the new version. Unfortunately, (for now) we can not give you any ETA of when it will be ready.

Kind regards!

richardeschloss commented 4 years ago

Thanks for the update. I appreciate the feedback.