Yes, hiding the popup is an appropriate thing to do.
It looks like your text values are referencing the data in the inputs instead of the member’s thing