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

Support the second price axis #129

Closed
kobermeit opened this issue Jul 17, 2019 · 9 comments · Fixed by #290
Closed

Support the second price axis #129

kobermeit opened this issue Jul 17, 2019 · 9 comments · Fixed by #290
Assignees
Labels
enhancement Feature requests, and general improvements.
Milestone

Comments

@kobermeit
Copy link

Is your feature request related to a problem? Please describe.

I'm drawing multiple line series and a candlestick series on a chart. I can see the price axis on the right, but I can't see the axis for the other series, which makes it hard to read. With my old ecahrts library, I had 3 axis on the chart (2 on the left, 1 on the right).

Describe the solution you'd like
Ability to set something like "showAxis: true" on each series and then configure if its on the left or right.

Additional context

Below is a screenshot of the current chart I have in eCharts. This is live and working, but I prefer the lightweight TV chart as the scrolling is far smoother than eCharts.

image

@timocov
Copy link
Contributor

timocov commented Jul 18, 2019

Did you see other our products to display charts https://www.tradingview.com/HTML5-stock-forex-bitcoin-charting-library/ ? There is "Technical Analysis Charts" and there is such feature (demo https://charting-library.tradingview.com/).

We don't plan to add them to lightweight-charts yet.

@kobermeit
Copy link
Author

@timocov Thanks, I actually have that TV charting library and currently using it in my product in another section. I just wanted to use lightweight for this chart instead and was hoping it would support this. But if the best solution is to use the full chart library, then that's fine :)

@timocov
Copy link
Contributor

timocov commented Jul 18, 2019

We want to keep lightweight-charts library lightweight 😉. Multiple axis feature is epic for the library and would add a lot of code and would increase the size of the lightweight-charts. Thus, unfortunately, I'm going to close this issue.

@timocov timocov closed this as completed Jul 18, 2019
@Tucsky
Copy link

Tucsky commented Jul 27, 2019

@timocov what about creating two "overlay series" on the same axis ? without showing that axis on the side, would that fit in the lightweight vision?

Exemple : I have a the price serie, sma, and 3 overlay series, I want to scale thoses 3 series on the same SCALE otherwise seeing them overlapping each other based on their respective range makes very low sense imo.
image

@timocov
Copy link
Contributor

timocov commented Aug 1, 2019

without showing that axis on the side, would that fit in the lightweight vision?

If I understand you correctly this is already done in the lightweight-charts.

@Tucsky
Copy link

Tucsky commented Aug 1, 2019

@timocov Well thats would be great, I do hope Im just missing a setting here :)

Let me try explain again: In other word thoses 3 volume series below aren't "locked" in the same volume scale (which actually makes sense because I neved told lightweight-chart to do so), I just called addLineSeries or addBarSeries without telling the chart to create those 3 on the same scale (that setting im looking for)

scaleissue

The green line shouldn't go above the red or red above green as I pan left or right because the top / bottom of the visible point changed

@kirchet
Copy link
Contributor

kirchet commented Aug 2, 2019

@Tucsky, for now, there is no way to add an ovelay series to the same scale as other overlay series. Probably we should add this possibility.

@Tucsky
Copy link

Tucsky commented Aug 2, 2019

@kirchet thank you for considering it! Im gonna wait patiently.
Thats an awesome lib really, keeping it simple is what makes it so fast and accessible, but thats the only downside ive seen so I had to try :)

@timocov timocov added the out of scope Out of scope for the project. No plans to implement this feature at the moment. label Aug 19, 2019
@timocov
Copy link
Contributor

timocov commented Dec 19, 2019

Good news everyone! We decided to add possibility to add the second price axis to the chart. If you have some specific requests for that until it isn't in the production - feel free to post your thought here.

@timocov timocov reopened this Dec 19, 2019
@timocov timocov added enhancement Feature requests, and general improvements. and removed out of scope Out of scope for the project. No plans to implement this feature at the moment. labels Dec 19, 2019
@timocov timocov added this to the 1.3 milestone Dec 19, 2019
@timocov timocov changed the title Support multiple price axis Support the second price axis Dec 19, 2019
@eugene-korobko eugene-korobko mentioned this issue Dec 20, 2019
3 tasks
@timocov timocov modified the milestones: 1.3, 1.4 Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests, and general improvements.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants