Fix Clipboard Fallback by Checking textArea.select Return Value

[tmdeveloper007]

Summary of What Has Been Done

Modified fallbackCopyToClipboard to check the return value of textArea.select(). Returns false immediately if selection fails.

Changes Made

• utils/clipboard.ts: Check textArea.select() return value; return false early if selection fails

Impact it Made

• Provides clearer failure signaling when copy operation cannot proceed
• Avoids unnecessary execCommand calls when selection is not possible

Closes #8848

Note: Please assign this PR to the tmdeveloper007 account.

[retenta-bot]

This pull request improves the clipboard fallback handling by checking the return value of textArea.select(). This change provides clearer failure signaling when the copy operation cannot proceed and avoids unnecessary calls to document.execCommand('copy'). This approach aligns with previous decisions made to enhance performance and reliability in clipboard operations. Thank you for your contribution!

[Aamod007]

I appreciate you looking into improving the clipboard fallback in \utils/clipboard.ts. However, I have a concern with the changes: the .select()\ method on \HTMLTextAreaElement\ returns \�oid\ (undefined), not a boolean indicating success. Because of this, \const selected = textArea.select(); if (!selected) return false;\ will always evaluate to false and the function will never reach \execCommand('copy'). Please double-check the MDN documentation for \HTMLTextAreaElement.select()\ and revise this approach. I'll have to request changes for now.

[github-actions]

⚠️ Hey @tmdeveloper007, this pull request has been inactive for 3 days. It will be automatically closed in 2 days if no further activity occurs.

If you are still working on this, please push your latest changes or leave a comment to keep it active.