fix(ui): show delay preview on button hover#2947
Draft
clawctrl1 wants to merge 5 commits into
Draft
Conversation
added 5 commits
March 31, 2026 15:15
…ity selected Fixes FreezingMoon#2246 ## Problem When hovering over the active unit, the cursor doesn't change to indicate that the unit is selected but no action is available (no ability selected). ## Solution Added cursor state changes in hex.ts: - When hovering active creature with no ability selected: cursor = 'progress' - When leaving the hex: cursor = 'default' This provides visual feedback that the unit is interactive but waiting for an ability selection. ## Changes - Added cursor 'progress' on onInputOver when hovering active creature - Reset cursor to 'default' on onInputOut Closes FreezingMoon#2246
Fixes FreezingMoon#2363 ## Problem Holding Ctrl+Meta+L in the pre-match screen for too long can result in multiple file picker dialogs opening, causing confusion and potential issues. ## Solution Added a trap flag `isLoadingLog` to prevent concurrent file picker calls: - Set flag to true when opening file picker - Set flag to false when file is loaded or dialog is cancelled - Reject new requests if already loading This prevents multiple file pickers from opening simultaneously. ## Changes - Added `isLoadingLog` flag at module level - Check flag before opening file picker - Reset flag on success, error, or cancel Closes FreezingMoon#2363
Fixes FreezingMoon#1099 ## Problem When a creature picks up a drop, there's no audio feedback, making the action feel less satisfying. ## Solution Added `game.soundsys.playSFX('sounds/upgrade')` call in the `pickup()` method to play a sound effect when a drop is collected. The 'upgrade' sound is appropriate for pickups as it's already used for ability upgrades and other positive events. ## Changes - Added sound effect playback in Drop.pickup() method - Uses existing 'upgrade' sound effect Closes FreezingMoon#1099
Fixes FreezingMoon#1481 ## Changes - A key: Opens Audio view (unchanged) - Shift+A: Cycles through audio modes (full -> sfx -> muted) - Updated tooltip to show keyboard shortcuts ## Implementation - Modified pressA() in hotkeys.ts to handle Shift+A for cycling audio modes - Updated updateAudioIcon() tooltip to include shortcut hints - Uses existing cycleAudioMode() function Closes FreezingMoon#1481
Fixes FreezingMoon#1470 ## Changes - Added hover event listeners to delay button - Shows hint preview when hovering delay button - Indicates that creature will move to end of queue ## Implementation - Added mouseenter/mouseleave handlers to btnDelay - Added showDelayPreview() method to interface - Shows hint with delay action description Closes FreezingMoon#1470
|
Someone is attempting to deploy a commit to the FreezingMoon Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
|
@clawctrl1 You've made a total mess of latest PRs. This one doesn't compile and later ones don't even include changed related to the bugs addressed. Please never ever include patches fixing other issues that are in other PRs, because it creates merge conflict nightmares; always work on different branches. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1470
Problem
Hovering the delay button did not indicate where the unit will land in the queue.
Solution
Changes
Closes #1470