Skip to content

Conversation

@sanderl
Copy link
Contributor

@sanderl sanderl commented Oct 7, 2019

Description of the changes:

  • This is a working implementation of Key Graph Features for Graphing Calculator. It is not complete.
  • Remaining work: Hooking up the KGF EquationEditBox, Error Handling, Rich Edit Styling, Fit and Finish

@HowardWolosky HowardWolosky added the graphing calculator Work items related to the graphing calculator feature. label Oct 8, 2019
Copy link
Contributor

@joseartrivera joseartrivera left a 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!

@ghost ghost added the needs author feedback label Oct 8, 2019
Copy link
Contributor

@joseartrivera joseartrivera left a 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.

Copy link
Contributor

@joseartrivera joseartrivera left a 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! 👍

joseartrivera
joseartrivera previously approved these changes Nov 11, 2019
joseartrivera
joseartrivera previously approved these changes Nov 12, 2019
@sanderl sanderl merged commit 442ed6a into microsoft:feature/GraphingCalculator Nov 12, 2019
@sanderl sanderl deleted the key-graph-features branch January 16, 2020 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

graphing calculator Work items related to the graphing calculator feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants