Block multiple file pickers from opening simultaneously#2799
Block multiple file pickers from opening simultaneously#2799ldeong wants to merge 1 commit intoFreezingMoon:masterfrom
Conversation
Fixes FreezingMoon#2363 Added a trap to prevent multiple file pickers when Ctrl+Meta+L is held down in the pre-match screen. Uses a filePickerActive flag similar to the throttling pattern in gamelog.ts saveFile function. 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 Bounty: 2 XTR Wallet: 0x742d35Cc6634C0532925a3b844Bc9e7595f0bEb
|
Someone is attempting to deploy a commit to the FreezingMoon Team on Vercel. A member of the Team first needs to authorize it. |
|
👋 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! 🙏 |
|
COMPLETE |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@ldeong I actually can't reproduce the bug anymore in Firefox. Will have to test soon in more places 🐻 |
|
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 |
|
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 |
2 participants
Description
Fixes #2363 by preventing multiple file pickers when Ctrl+Meta+L is held down.
Changes
filePickerActiveflag to track picker stateTesting
Related Issues
Fixes #2363
Bounty: 2 XTR
Wallet: 0x742d35Cc6634C0532925a3b844Bc9e7595f0bEb