Skip to content

Conversation

@Amoursol
Copy link
Contributor

@Amoursol Amoursol commented Oct 25, 2022

Purpose

Adding in the ability to increase or decrease the text font size in the Python editor, by using CTRL and their middle-mouse scroll.

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated

Release Notes

(FILL ME IN) Brief description of the fix / enhancement. Mandatory section

Reviewers

@DynamoDS/dynamo

FYIs

@QilongTang

@QilongTang QilongTang added this to the 2.17.0 milestone Oct 25, 2022
@QilongTang
Copy link
Contributor

@Amoursol The clean-up commit is not clean, please fix the build error.. FYI: you cant rename the event hanlder without updating the reference in UI code

private const double FONT_MAX_SIZE = 60d;
private const double FONT_MIN_SIZE = 5d;

// Update function, increases/decreases by a specific increment
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. This comment doesn't really make sense.
  2. public methods should use ///summary comments.
  3. Why is this public?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mjkkirschner it's code from elsewhere ported. For sure not ideal for Dynamo. @QilongTang is fixing it for me 😊

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh missed this one, I will update this in another PR

@QilongTang QilongTang merged commit bb2b672 into master Oct 25, 2022
@QilongTang QilongTang deleted the pythonEditorTextZoom branch October 25, 2022 20:55
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.

4 participants