WIP: fix multi-pane feature

[ntf]

Type of PR: enhancement

PR checklist:

• Addresses an existing issue: fixes Multiple panes support #50
• Includes tests
• Documentation update

Overview of change:
This is base on the pane widget and pane separator feature previously implemented in 22c6b03.

Is there anything you'd like reviewers to focus on?
API design. New API added so far (To be finalized)

Things I have implemented so far:

• series option to configure which pane a series should be rendered ( pane: number to SeriesOptionsCommon )
• ChartApi#removePane(index: number) so that you can remove a pane (and series on it) programmatically.
• ChartApi#swapPane(first: number, second: number) for swapping the positions of the two pane.
• code to handle series.applyOptions( { pane: newIndex } );
• ChartApi#getPaneElements() for getting container elements to each pane. (I need the container to draw legends to each pane)

Things I am also implementing but not going to be included in this PR:

• React binding with pane and legend supports

Demo here (It maybe outdated): https://jsfiddle.net/adrianntf/6qea5ytv/2/

You can use below command to install my version to your project locally:

yarn add git+https://github.com/ntf/lightweight-charts.git
or npm install git+https://github.com/ntf/lightweight-charts.git

You should not use this directly in your production environment. However, you may review my code and merge into your own folk of the project.

[ntf]

Things I have implemented so far:

• series option to configure which pane a series should be rendered ( pane: number to SeriesOptionsCommon )
• ChartApi#removePane(index: number) so that you can remove a pane (and series on it) programmatically.
• ChartApi#swapPane(first: number, second: number) for swapping the positions of the two pane.
• code to handle series.applyOptions( { pane: newIndex } );
• ChartApi#getPaneElements() for getting container elements to each pane. (I need the container to draw legends to each pane)

Things I am also implementing but not going to be included in this PR:

• React binding with pane and legend supports

[ntf]

resolved merge conflicts with tradingview's master branch

[qwpto]

This PR still merges easily to the master, qwpto@7efe1bc it would be a great feature to have instead of needing to maintain a fork of the project to get it. @ntf how did you generate the inlined code for the jsfiddle?

[FairyWorld]

resolved merge conflicts with tradingview's master branch

There is a conflict with the latest version. I hope you can solve it

[carere]

Does this feature still in the loop for future release ?
@qwpto Where is the fork with this feature implemented ?

[qwpto]

Does this feature still in the loop for future release ? @qwpto Where is the fork with this feature implemented ?

@carere There's a pull request link in my comment above where it can be found.

[rwitzlib]

Here is an updated fork people can reference if they'd like: https://github.com/rwitzlib/lightweight-charts
Currently updated to v4.1.3 but Ill try to keep this updated until multi-pane is officially supported

[carere]

@rwitzlib Can you explain to me how can I add another pane to the chart with your fork ?

[rwitzlib]

@rwitzlib Can you explain to me how can I add another pane to the chart with your fork ?

Of course, I left a comment here: #50 (comment) so its a bit easier for people to see