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

Add support for rewinding the chart's prediction while viewing current data #129

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

SHewitt95
Copy link
Contributor

@SHewitt95 SHewitt95 commented May 27, 2020

Description

Adds the ability to see how the chart looked on certain days and times, depending the prices available up to those days and times. The line drawn for "Daily Price" retains its shape as the rest of the chart shifts to previous states.

Demo (gif)

acnh

Open Questions

  • How do we feel about the errors that Material UI generates in the console? Here's what I ran into, and it seems the modal that warns about clearing data triggers the same error.

@elxris
Copy link
Owner

elxris commented May 30, 2020

Thank you so much for the PR. However, I feel like this kind of "advanced" feature should be not enabled for default.
I was thinking about making a special configuration section to enable as many features as you want. What do you think about this?

Regarding this PR I feel like a Slider suits best this feature than the dropdown.

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

Successfully merging this pull request may close these issues.

[FEATURE] Way to rewind the prediction point while up-to-date data is in place
2 participants