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.

Sign up for GitHub

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

Jump to bottom

Multiple series: error while trying to scroll the chart #373

Closed
Jeronimotio opened this issue Mar 8, 2020 · 2 comments · Fixed by #424
Closed

Multiple series: error while trying to scroll the chart #373

Jeronimotio opened this issue Mar 8, 2020 · 2 comments · Fixed by #424
Assignees
@timocov
Labels
bug Unexpected problem or unintended behavior.
Milestone
3.0

Comments

@Jeronimotio
Copy link

Lightweight Charts Version: 2.0.0

Steps/code to reproduce:
Try to scroll example chart with multiple series to the past

Actual behavior:
Type Error: Cannot read property 'x' of undefined

Expected behavior:
No throw

CodeSandbox/JSFiddle/etc link:
https://codesandbox.io/s/tender-golick-lj8tb
@timocov timocov added the bug Unexpected problem or unintended behavior. label Mar 10, 2020
@timocov timocov added this to the 3.0 milestone Mar 10, 2020
@timocov timocov mentioned this issue Apr 3, 2020
Closed
@kuamatzin
Copy link

Hi! Any fix?

@timocov
Copy link
Contributor

timocov commented Apr 16, 2020

@kuamatzin We'll fix it soon (in 3.0).

timocov added a commit that referenced this issue Apr 30, 2020
@timocov
Fixed rendering area when it's out of viewport
b82c113 
Fixes #373
@timocov timocov self-assigned this Apr 30, 2020
@timocov timocov mentioned this issue Apr 30, 2020
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timocov timocov

Labels
bug Unexpected problem or unintended behavior.
Projects
None yet
Milestone
3.0
Development

Successfully merging a pull request may close this issue.

Fixed rendering area when it's out of viewport tradingview/lightweight-charts
3 participants
@timocov @Jeronimotio @kuamatzin