-
Notifications
You must be signed in to change notification settings - Fork 162
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
Creating an off_focus
hook in TextBox2D
#501
Conversation
on_blur
hook in TextBox2D
on_blur
hook in TextBox2D
Codecov Report
@@ Coverage Diff @@
## master #501 +/- ##
==========================================
+ Coverage 89.05% 89.14% +0.08%
==========================================
Files 54 54
Lines 11534 11541 +7
Branches 1130 1131 +1
==========================================
+ Hits 10272 10288 +16
+ Misses 962 957 -5
+ Partials 300 296 -4
|
Please use |
A test is missing too. |
on_blur
hook in TextBox2D
off_focus
hook in TextBox2D
Hey @skoudoro , |
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.
Thank you for this PR @ganimtron-10.
Looks good to me. merging
To know when the user has completed editing in the
TextBox2D
, creating anon_blur
hook.Required in #499 .
Also changing the condition sequence in the
handle_character
function, because when we press "return" key instead of removing focus from the UI it adds the "\r" character to the message.