Skip to content

Conversation

@Bennethxyz
Copy link

This PR prevents multiple file picker dialogs from opening when holding Ctrl+Meta+L on the pre-match screen.

  • Introduces a throttled wrapper around the existing log file loader using underscore's throttle (same pattern used for saving logs in utility/gamelog.ts).
  • Keeps behavior identical otherwise; only one dialog can be triggered per second.
  • Adds minor safety checks for empty selection and types.

Tested locally by holding the hotkey; only a single chooser appears.

Fixes #2363.

@vercel
Copy link

vercel bot commented Nov 3, 2025

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

A member of the Team first needs to authorize it.

@Bennethxyz
Copy link
Author

Also addresses #2174 by flashing the cancel icon on unusable/used active ability clicks.

@vercel
Copy link

vercel bot commented Nov 4, 2025

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

Project Deployment Preview Updated (UTC)
ancientbeast Ready Ready Preview Nov 4, 2025 4:58am

@DreadKnight
Copy link
Member

DreadKnight commented Nov 4, 2025

@Bennethxyz As I was saying in the other PR for this, can't currently reproduce the issue myself; will try different browsers and operating systems pretty soon. I like that you took a different approach altogether though.

Also addresses #2174 by flashing the cancel icon on unusable/used active ability clicks.

Regarding this one, wish it was a different PR altogether to keep things nicer.
I've tested it and works fine overall, but found an issue:
no cancel icon is displayed when clicking the passive ability after you materialize and unit with Dark Priest.

@DreadKnight DreadKnight marked this pull request as draft November 4, 2025 01:47
@Bennethxyz Bennethxyz marked this pull request as ready for review November 7, 2025 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

block multiple file pickers [bounty: 2 XTR]

2 participants