fix(sqllab): Skip AceEditor in inactive tabs

[justinpark]

SUMMARY

Since destroyInactiveTabPane is disabled for improving rendering perf on switching tabs in #26791, I have confirmed that the Ace Editor was monitoring and responding to events even on inactive tabs.
As a result, race conditions related to the cursor position were occurring when switching between tabs. This commit addresses these issues by modifying the behavior so that, similar to destroyInactiveTabPane, the Ace Editor component is excluded when a tab becomes inactive.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before:

before--inactive-ace-editor.mov

Active:

after--inactive-ace-editor.mov

TESTING INSTRUCTIONS

Specs are added

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[michael-s-molina]

@justinpark Isn't possible to just make the ace editor stop listening to events when inactive instead of removing it from the DOM? That way you would retain the previous performance gains.

[justinpark]

@michael-s-molina That approach involves many aspects that need to be checked, such as regression, making it challenging to implement as a hotfix.
It would be better to revisit and resolve the issue related to the performance of the Ace Editor (if we feel downgrade) after addressing the current problem.