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

Navigator xAxis Id not set correctly #20382

Closed
cnighswonger opened this issue Jan 4, 2024 · 1 comment · Fixed by #21040
Closed

Navigator xAxis Id not set correctly #20382

cnighswonger opened this issue Jan 4, 2024 · 1 comment · Fixed by #21040

Comments

@cnighswonger
Copy link

cnighswonger commented Jan 4, 2024

Expected behaviour

Per the API docs, one should be able to set the navigator xAxis id and then retrieve the navigator xAxis object using that id. In the absence of a user-defined id, the id should default to undefined.

Actual behaviour

The call to chart.get() on a user defined id returns undefined.

Furthermore, the id defaults to navigator-x-axis rather than undefined.

Live demo with steps to reproduce

Fiddle displaying buggy behaviour.

Fiddle showing that the navigator xAxis id is not set per configuration (or per docs for that matter)

Fiddle showing that the navigator xAxis can be retrieved using chart.get('navigator-x-axis')

Product version

Highstock JS v11.2.0 (2023-10-30)

Affected browser(s)

Firefox 121.0 Windows 10

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Jan 4, 2024
@cnighswonger cnighswonger changed the title Navigator Id not set correctly Navigator xAxis Id not set correctly Jan 4, 2024
@karolkolodziej
Copy link
Contributor

karolkolodziej commented Jan 8, 2024

Hi @cnighswonger!
Thank you for reporting this and preparing all the demos!

Internal note:
The id is being hardcoded. We should use userOptions if defined.

id: 'navigator-x-axis',

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.

3 participants