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