Skip to content
This repository has been archived by the owner on Jul 10, 2020. It is now read-only.

Refactoring to use Map to build chart options and data. #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dyorgio
Copy link

@dyorgio dyorgio commented Dec 7, 2016

Stop to work with elemental json lib and start to use plain java.util.Map because elemental don't support a function or raw js object in JsonValue.

Implemented userCallback in Ticks.

#39
#38 (initial work)

….Map because elemental don't support a function or raw js object in JsonValue.

Implemented userCallback in Ticks.
@moberwasserlechner
Copy link
Owner

moberwasserlechner commented Dec 8, 2016

I appreciate your effort but It would have been nice if you first discussed your planned changes here rather than changing half of code base. Anyway I'll try to review it on the weekend.

@dyorgio
Copy link
Author

dyorgio commented Dec 8, 2016

@moberwasserlechner, you are right, I think a lot before start to do this modification, it is really impact in all codebase. But, after try many times to make it works with elemental (because I need to pass a JS function to chart) I only found this solution.
I believe that this functionality is very important to support customizations and plugins.
Tks for getting time to review it :).

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

Successfully merging this pull request may close these issues.

2 participants