Show/Hide efficiency update.
I just realised that _toggleShow()
isn't the most efficient. It toggles the show/hide status on fields/groups and then if it's being hidden, nullifies the fields it contains regardless of whether or not it did anything.
I propose that we only nullify if something was actually hidden.