-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Key graph features #704
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
Key graph features #704
Conversation
…ut all strings into the Equation object
…roperties in Equation object
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.
Nice work! Thanks for digging into this feature! Looking good so far, I left a few comments for you to consider. Thanks again!
src/Calculator/Views/GraphingCalculator/GraphingCalculator.xaml.cpp
Outdated
Show resolved
Hide resolved
src/Calculator/Views/GraphingCalculator/KeyGraphFeaturesPanel.xaml.cpp
Outdated
Show resolved
Hide resolved
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.
Nice iterations, looks like we are getting pretty close! I have not yet done a buddy build and taken a look at the UI but I can do that on my next pass if you'd like.
… TemplateSelector
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.
Looking good! The new UI design is 🔥. Found a few bugs that shouldn't be too hard to fix, although the root cause is unclear for a few. Let me know what you think! 👍
…eColor.Text resource that was mistakenly taken out
…ion populated in a new equationtextbox after the previous one was deleted
Description of the changes: