fix: LEAP-1586: Fix CommentsOverlay display without preloading image functionality

[Gondragos]

Apparently, the conditions we used do not work without feature flag fflag_feat_front_lsdv_4583_6_images_preloading_short.

But it also seems that this condition made sense only at the stage of developing the CommentsOverlay functionality. Somehow, I can't find a reason why I needed it before. So I'm removing it for the greater good.

PR fulfills these requirements

• Tests for the changes have been added/updated (for bug fixes/features)
• Docs have been added/updated (for bug fixes/features)
• 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)

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

Describe the reason for change

Comments overlay is not displayed with some feature flags states.

What alternative approaches were there?

We could change the condition to

if (isFF(FF_LSDV_4583_6) && tag.imageIsLoaded === false) return false;

but that leads to the blinking of all comment icons when switching pages for multi-item object tags.

Does this PR introduce a breaking change?

• Yes, and covered entirely by feature flag(s)
• Yes, and covered partially by feature flag(s)
• No
• Not sure (briefly explain the situation below)

What level of testing was included in the change?

• e2e
• integration
• unit

Which logical domain(s) does this change affect?

CommentsOverlay

[netlify]

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

Name	Link
🔨 Latest commit	114bf7e
🔍 Latest deploy log	https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/670ee42576a70d00080259aa

[netlify]

✅ Deploy Preview for heartex-docs canceled.

Name	Link
🔨 Latest commit	114bf7e
🔍 Latest deploy log	https://app.netlify.com/sites/heartex-docs/deploys/670ee4255b59fd0008b7015b