-
Notifications
You must be signed in to change notification settings - Fork 2
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
[WEB] PR #102 Rearrange Charts #282
Conversation
Thanks @fyang151! @JordanChen123 do you mind doing a pass and taking a look at this? |
Great work! The overall code looks good to me. There is a slight issue with the state of the graphs after refreshing the page. As an example, if you move the speed graph to the very bottom, and then refresh the page, the state of the Drag and Drop menu stays the same as it was before refreshing the page, but the graphs go back to where they were when the page was loaded up initially. |
oh oops ill look at it rn and try to get it done by tn meeting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Fred! Also, dropping some minor comments dealing with style/structure.
Description
Verification
Resources