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

Update y-axis range when x-axis rangeslider is used #2439

Closed
IanWorthington opened this issue Mar 5, 2018 · 5 comments
Closed

Update y-axis range when x-axis rangeslider is used #2439

IanWorthington opened this issue Mar 5, 2018 · 5 comments

Comments

@IanWorthington
Copy link

There's some JS code over here:

plotly/plotly.R#912

to autoupdate the yaxis as the xaxis changes.

There's also some discussion around this maybe not being a better default than the current not scaling at all, but also a request to open the issue here, which I can't see was ever done, so I am.

I'm sure it's not a perfect solution to all cases but can it be added in as an option maybe?

@etpinard
Copy link
Contributor

etpinard commented Mar 5, 2018

I'm not 100% sure what you're requesting.

You might want to look at current PR #2364 which is adding a few rangeslider+yaxis configuration options.

@IanWorthington
Copy link
Author

Thanks for the link. The code in plotly/plotly.R#912 adds automatic rescaling of the y-axis as the rangeslider is adjusted, which, iiuc, I don't think is in the PR?

@alexcjohnson
Copy link
Collaborator

@IanWorthington Is your intent that on adjusting the range slider, the y axis autoranges to just the data that's visible in the new x range? That's a feature we've had on the list for quite some time... #1876 (comment)

@IanWorthington
Copy link
Author

@alexcjohnson Yes, that looks like it might be it. Do I take it node of that is implemented yet?

@alexcjohnson
Copy link
Collaborator

Correct, that issue is to collect the behavior we want to add, and make sure that it's all mutually consistent and compatible, but it's not yet implemented.

Closing this issue, lets continue the discussion over there.

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

No branches or pull requests

3 participants