Fix input disabling

[scalvert]

Fixes the input disabling where using data caused it to not work!

Status: Open for visibility

Reviewers: @tedtate @kpeatt @Helen-Mobify

Changes

• fixed enabling/disabling inputs
• added checks to ensure these methods only run once if a pinny is open/lockup is locked.

  How to test-drive this PR

• checkout branch
• grunt test
• grunt serve and ensure that pinny is correctly enabling/disabling inputs on open/close

  TODO

• [ ] Write passing test for tab index storage on open
• [ ] Write passing test for tab index restoring on close

[scalvert]

@Helen-Mobify

[kpeatt]

We'll need to make this change to Pikabu as well.

[wizardlyhel]

The document.activeElement can be undefined if there is an iframe in the pinny.

[scalvert]

OK I've done some rework with things like method names and also order of operations. I needed to tweak the order of operations of input disabling to ensure it was running at the correct time.

[wizardlyhel]

:+1: looks good @scalvert