fix: OPTIC-765: TypeError: Cannot read properties of null (reading 'getBoundingClientRect')

[bmartel]

PR fulfills these requirements

• Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
• Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
• Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

(check all that apply)

• Product design
• Backend (Database)
• Backend (API)
• Frontend

What does this fix?

The Tooltip in certain circumstances could throw an error trying to align the position of an already unmounted component which threw the error in the PR title. Added a check to ensure this does not attempt to recalculate the position of the tooltip if either the trigger or tooltip component has unmounted from the DOM during the async transition. Also I cleaned up the deps arrays to remove any stable refs from here as it is not required.

Notes to reviewers

I was not able to reproduce this in any of the Tooltip usages I was able to determine. The Sentry stack trace points directly to this function being the culprit, and given the fact we have these async functions happening within useEffects without the guard proposed in this PR, it aligns with the type of behaviour that would result in this exact error.

[netlify]

✅ Deploy Preview for label-studio-docs-new-theme canceled.

Name	Link
🔨 Latest commit	32a699f
🔍 Latest deploy log	https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/6699716c641c440008d6dd6c

[netlify]

✅ Deploy Preview for heartex-docs canceled.

Name	Link
🔨 Latest commit	32a699f
🔍 Latest deploy log	https://app.netlify.com/sites/heartex-docs/deploys/6699716ce8bc5e00083d13dc