AncientBeast icon indicating copy to clipboard operation
AncientBeast copied to clipboard

Block multiple file pickers from opening simultaneously

Open ldeong opened this issue 2 months ago • 7 comments

Description

Fixes #2363 by preventing multiple file pickers when Ctrl+Meta+L is held down.

Changes

  • Added filePickerActive flag to track picker state
  • Block new pickers if one is already open
  • Reset flag on file selection, error, or cancel
  • Safety timeout after 5 seconds
  • Follows the same pattern as the throttling in gamelog.ts

Testing

  • Tested holding Ctrl+Meta+L - only one picker opens
  • File selection works normally
  • Cancel works and resets flag
  • Safety timeout prevents stuck state

Related Issues

Fixes #2363


Bounty: 2 XTR
Wallet: 0x742d35Cc6634C0532925a3b844Bc9e7595f0bEb

ldeong avatar Oct 06 '25 06:10 ldeong

Someone is attempting to deploy a commit to the FreezingMoon Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Oct 06 '25 06:10 vercel[bot]

👋 Hi! This PR is ready for review. It fixes the issue described and includes the necessary changes. Could you please review when you have a moment? Thanks! 🙏

ldeong avatar Oct 06 '25 08:10 ldeong

COMPLETE

ldeong avatar Oct 06 '25 10:10 ldeong

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
ancientbeast Ready Ready Preview Oct 6, 2025 5:47pm

vercel[bot] avatar Oct 06 '25 17:10 vercel[bot]

@ldeong I actually can't reproduce the bug anymore in Firefox. Will have to test soon in more places 🐻

DreadKnight avatar Oct 06 '25 17:10 DreadKnight

Hi! Following up on this PR. I've addressed the file picker issue to prevent multiple instances from opening simultaneously. Happy to make any adjustments based on your review. Looking forward to your feedback! 🎯

Wallet for bounty payment: 0x30eceb0728964a0b07dd0cac20bc7afbe723253c

ldeong avatar Oct 14 '25 21:10 ldeong

Hi! Just following up on this PR. Ready for review when you have time. Let me know if any changes are needed. Thanks! 🙏

Wallet for bounty payment: 0x30eceb0728964a0b07dd0cac20bc7afbe723253c

ldeong avatar Oct 14 '25 21:10 ldeong