- Notifications
You must be signed in to change notification settings - Fork112
Correctly Lose Focus (finish CETV#14)#229
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
tom-ludwig left a comment
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.
LGTM!
You probably know this, but you can use the trailing closure syntax on thesorted method.
0xWDG left a comment
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.
LGTM
thecoolwinter commentedFeb 4, 2024
Oh, very true! Didn't catch that. |
Description
Finishes the required changes to makeCodeEditTextView#14 work in CESE, once those changes are merged with a package update.
Removes the first responder acquisition when updating cursors, the text view should handle focus state and does so correctly on the main branch. Also fixes a small ordering bug with a check variable related to this issue.
Related Issues
Checklist
Screenshots
N/A