fix: add error logging to empty catch block in keymap.ts (@Dawn-Fighter)

[Dawn-Fighter]

Description

Fixes an empty catch block in the flashKey function that was silently swallowing animation errors.

Changes

• Added error logging to the catch block at line 139 in frontend/src/ts/elements/keymap.ts
• Matches the existing error handling pattern used elsewhere in the same file (line 95-98)
• Uses console.log for consistency with the codebase's internal logging style

Type of Change

• Bug fix (non-breaking change which fixes an issue)

Code Change

Before:

} catch (e) {}

After:

} catch (e) {
  if (e instanceof Error) {
    console.log("could not flash keymap key: " + e.message);
  }
}

Testing

• Code follows the existing pattern in the same file (line 95-98)
• Change is consistent with TypeScript best practices
• Uses type guard (instanceof Error) before accessing error properties

Additional Context

This change improves debugging by logging animation errors instead of silently ignoring them, making it easier to diagnose issues with keymap flash animations.

[Copilot]

Pull request overview

This PR improves error handling in the keymap.ts file by adding proper error logging to a previously empty catch block in the flashKey function. This ensures that animation errors are logged instead of being silently swallowed, improving debuggability.

Changes:

• Added error logging with an appropriate error message to the catch block in the flashKey function

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Miodec]

Not sure why is this needed?