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

Last scatter point not visible #19243

Closed
krp12345 opened this issue Jun 28, 2023 · 6 comments · Fixed by #20282
Closed

Last scatter point not visible #19243

krp12345 opened this issue Jun 28, 2023 · 6 comments · Fixed by #20282

Comments

@krp12345
Copy link

Expected behaviour:
All scatter point should be visible when chart is loaded.
Please check:
https://www.highcharts.com/forum/viewtopic.php?f=12&t=51074&e=1&view=unread#unread

Actual behaviour:
Last scatter point is not getting visible when graph is loaded.

Affected browser: chrome, not sure about other browser.

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Jun 28, 2023
@jakubjanuchta
Copy link
Contributor

jakubjanuchta commented Jun 29, 2023

Internal note:
The last point gets loaded in v9.1.2, https://jsfiddle.net/BlackLabel/q42yo97h/.

@krp12345
Copy link
Author

10.2.1 version does not show last point.

Please check : jsfiddle.net/3ecjgust

@bm64
Copy link
Member

bm64 commented Jul 2, 2023

As it was suggested in forum topic, overscroll property can be used as a workaround: https://jsfiddle.net/BlackLabel/nc7jwbkt/

@krp12345
Copy link
Author

krp12345 commented Jul 4, 2023

@pawelfus
Copy link
Contributor

The last scatter point issue has been fixed and is available on the master branch: https://jsfiddle.net/BlackLabel/q42yo97h/2/

The second part of the issue is still unresolved: https://jsfiddle.net/BlackLabel/6ukj0t8w/1/ vs https://jsfiddle.net/BlackLabel/6ukj0t8w/

@pawellysy
Copy link
Member

pawellysy commented Dec 5, 2023

Internal note
This happens because the scatter points are not taken into consideration, when calculating ordinalPositions because of the property takeOrdinalPosition which is defined here: https://github.com/highcharts/highcharts/blob/master/ts/Series/Scatter/ScatterSeries.ts#L168

The line mentions #2342

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

Successfully merging a pull request may close this issue.

6 participants