-
Notifications
You must be signed in to change notification settings - Fork 357
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
Fixed lost focus on code editor after first state update. #6249
Conversation
@Hyperkid123 Cannot apply the following label because they are not recognized: ivanchuk yes |
Checked commit Hyperkid123@f9050b7 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 |
@miq-bot add_label ivanchuk/yes |
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.
Tested in UI, it works great! 🥇
Fixed lost focus on code editor after first state update. (cherry picked from commit fced5af) https://bugzilla.redhat.com/show_bug.cgi?id=1754543
Ivanchuk backport details:
|
fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1754543
The issue here was with passing anonymous function to
FieldProvide
. It did not have any reference to it and was creating new instance of that component instead of updating its props. That caused loosing the focus of the field. By wrapping the children and naming the component the issue is fixed.I also removed the value from state and let parent component handle it.