Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot create custom plugins by extending Plugin class #1665

Closed
creage opened this issue Sep 9, 2020 · 3 comments
Closed

Cannot create custom plugins by extending Plugin class #1665

creage opened this issue Sep 9, 2020 · 3 comments

Comments

@creage
Copy link

creage commented Sep 9, 2020

v2 is a TS code now, and there is no JS version of Plugin class

One of my charts has a custom plugin, that is built based on the Plugin class of billboard.

import Plugin from 'billboard.js/src/plugin/Plugin';

export default class CustomPlugin extends Plugin {
...
}

This code does not work anymore. Yes, I can import TS version, but that will require adding a TS compiler to my build, which I would like to avoid.

@netil
Copy link
Member

netil commented Sep 9, 2020

I see. Let me figure out for any solution on this.

netil added a commit to netil/billboard.js that referenced this issue Sep 11, 2020
Plugin dev from the  JS based project could face extra work
because of the TS based Plugin class.
So, providing JS version will solve the issue.

Ref naver#1665
@netil netil closed this as completed in 4a20480 Sep 11, 2020
netil pushed a commit that referenced this issue Sep 11, 2020
# [2.1.0-next.3](2.1.0-next.2...2.1.0-next.3) (2020-09-11)

### Bug Fixes

* **interaction:** fix for data point click on mobile ([727c26f](727c26f)), closes [#1651](#1651)
* **interaction:** fix retrieving event rect position ([6fc449e](6fc449e)), closes [#1670](#1670)
* **shape:** fix possible condition removal by transpiler ([781fb61](781fb61)), closes [#1663](#1663)

### Features

* **plugin:** Add JS  Plugin class file ([4a20480](4a20480)), closes [#1665](#1665)
@creage
Copy link
Author

creage commented Sep 11, 2020

Well, having it as a ES6 class needs it to be compiled first, and usually node_modules are excluded from compilations.

Can we have a compiled version of that plugin class in the /dist/ folder please?

netil pushed a commit that referenced this issue Sep 25, 2020
# [2.1.0](2.0.3...2.1.0) (2020-09-25)

### Bug Fixes

* **api:** prevent TypeError when chart already destroyed ([ce42768](ce42768)), closes [#1613](#1613)
* **axis:** fix y axis clip-path coordinates ([bdf7694](bdf7694)), closes [#1572](#1572)
* **bar:** Fix not firing data.onclick ([62e7a10](62e7a10)), closes [#1619](#1619) [#1620](#1620)
* **bar:** fix on bar tooglip grouped false ([4bd69be](4bd69be)), closes [/github.com//issues/1663#issuecomment-690991679](https://github.com//github.com/naver/billboard.js/issues/1663/issues/issuecomment-690991679)
* **grid:** Fix grid text position update ([056b565](056b565)), closes [#1592](#1592)
* **interaction:** bind touch event only for rect element ([12da3e4](12da3e4)), closes [#1650](#1650)
* upgrade d3-ease from 1.0.6 to 1.0.7 ([#1617](#1617)) ([4f78533](4f78533))
* **interaction:** correct draggable selection functionality ([160f873](160f873)), closes [#1642](#1642)
* upgrade d3-brush from 1.1.5 to 1.1.6 ([#1605](#1605)) ([624f044](624f044))
* **interaction:** fix for data point click on mobile ([727c26f](727c26f)), closes [#1651](#1651)
* **interaction:** fix retrieving event rect position ([6fc449e](6fc449e)), closes [#1670](#1670)
* **legend:** Fix error for color.threshold option ([6e06629](6e06629)), closes [#1604](#1604) [#1611](#1611)
* **legend:** Make legend blurry after click ([319d608](319d608)), closes [#1599](#1599)
* **shape:** fix possible condition removal by transpiler ([781fb61](781fb61)), closes [#1663](#1663)
* **step:** Fix rendering nullish data ([dbe7b9b](dbe7b9b)), closes [#1637](#1637)
* **types:** Fix donut option types ([1917169](1917169)), closes [#1615](#1615)
* **types:** Specify context for callbacks ([f3b9f26](f3b9f26)), closes [#1551](#1551)

### Features

* **area:** Intent to ship area.front ([fe315dc](fe315dc)), closes [#1543](#1543)
* **axis:** Improve log scale to handle 0 ([ca6cf62](ca6cf62)), closes [#1578](#1578)
* **axis:** Intent to ship axis.y2.type ([a94c25e](a94c25e)), closes [#1575](#1575)
* **interaction:** avoid multiple <rect> generation ([97df63a](97df63a)), closes [#1642](#1642)
* **plugin:** Add JS  Plugin class file ([4a20480](4a20480)), closes [#1665](#1665)
* **theme:** Add new 'datalab' theme ([aba20d4](aba20d4)), closes [#241](#241)
@mayfield
Copy link

mayfield commented May 1, 2022

Could really use this too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants