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

Button All rangeSelector from default array of configuration objects, don't work properly. #16319

Closed
hihajdus opened this issue Sep 10, 2021 · 3 comments · Fixed by #16429
Closed

Button All rangeSelector from default array of configuration objects, don't work properly. #16319

hihajdus opened this issue Sep 10, 2021 · 3 comments · Fixed by #16429
Assignees
@karolkolodziej
Labels
Product: Highcharts Stock Type: Bug Type: Regression

Comments

@hihajdus
Copy link
Contributor

Expected behaviour

The rangeSelector button All should show all range after click all second time.

Actual behaviour

Button All rangeSelector from default array of configuration objects, don't work properly for view all range.
Didn't show all range after double click.

Live demo with steps to reproduce

https://jsfiddle.net/BlackLabel/8wfzot5g/

stock

Product version

Highstock JS v9.2.2 (2021-08-24)
The problem doesn't occur with versions up to 9.1 and earlier.

Affected browser(s)

All
@KacperMadej
Copy link
Contributor

Internal note:
Doesn't show on a large chart. Fixed width always shows the problem: https://jsfiddle.net/BlackLabel/8wfzot5g/1/

@karolkolodziej
Copy link
Contributor

Internal note:
Disabling dataGrouping solves the flicking. I guess the debug should start from this.
Demo: https://jsfiddle.net/BlackLabel/2uL7qeoh/

@karolkolodziej karolkolodziej self-assigned this Sep 28, 2021
@karolkolodziej
Copy link
Contributor

Internal note:
Direct cause: #16241

karolkolodziej added a commit that referenced this issue Sep 30, 2021
@karolkolodziej
Fixed #16319, initial commit for a visual test.
088505d
@karolkolodziej karolkolodziej mentioned this issue Sep 30, 2021
Merged
TorsteinHonsi pushed a commit that referenced this issue Oct 5, 2021
@karolkolodziej @TorsteinHonsi
Fixed #16319, initial commit for a visual test.
99a22e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@karolkolodziej karolkolodziej

Labels
Product: Highcharts Stock Type: Bug Type: Regression
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bugfix/16319-rangeSelector-all-data-grouping highcharts/highcharts
4 participants
@KacperMadej @hihajdus @highsoft-bot @karolkolodziej