-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Convert TermControl to .xaml #4729
Conversation
This commit removes all of the custom UI initialization code in TermControl and replaces it with a xaml file. Some dead or reundant code was removed as part of this refactoring. It also fixes two (quasi-related) issues: * The search box, on first launch, was offset by the scrollbar even if the scrollbar was invisible. * The scrollbar state wasn't hot-reloadable.
To be considered: All of the getters ( |
This broke Background color (!) |
The xaml compiler automatically generates weak event handlers for every event we subscribe to using xaml. It automatically connects the local members of the TermControl to the Names specified in the xaml file. It even emitted Unload handlers for the search box control because I made it dynamically loadable. |
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.
Nothing worth blocking over. Excited for this to go in!!!
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 for translating the code into XAML 😄, and everything seems to work just as expected, so 🚢 it
This pull request is like #4729 but for TSFInputControl.
This commit removes all of the custom UI initialization code in
TermControl and replaces it with a xaml file. Some dead or reundant code
was removed as part of this refactoring.
It also fixes two (quasi-related) issues:
the scrollbar was invisible.
PR Checklist
Validation Steps Performed
Launched, opened, closed, searched and TSF-inputted-into controls.
Fixes #4920