Skip to content

fix: click events for debug menu toggle in the settings page#1859

Merged
Bilb merged 3 commits intodevfrom
fix/settings_debug_toggle
Feb 23, 2026
Merged

fix: click events for debug menu toggle in the settings page#1859
Bilb merged 3 commits intodevfrom
fix/settings_debug_toggle

Conversation

@Aerilym
Copy link
Collaborator

@Aerilym Aerilym commented Feb 23, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 23, 2026 23:13
@Aerilym Aerilym requested review from Bilb and removed request for Copilot February 23, 2026 23:15
Copilot AI review requested due to automatic review settings February 23, 2026 23:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes click event handlers for the debug menu toggle in the settings page. The previous implementation incorrectly used keyboard event handlers as click handlers, passing null as any as a fake event. This PR properly separates click and keyboard handlers and enhances the debug mode toggle to support both enabling and disabling, with user feedback via toast notifications.

Changes:

  • Refactored event handlers to properly separate onClick and onKeyDown logic
  • Fixed debug mode toggle to support both enabling and disabling (previously only enabled)
  • Added toast notification feedback when toggling debug mode
  • Fixed click counter logic using functional setState to avoid stale closures

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

@Aerilym Aerilym requested a review from Bilb February 23, 2026 23:55
@Bilb Bilb merged commit 40c6870 into dev Feb 23, 2026
10 checks passed
@Bilb Bilb deleted the fix/settings_debug_toggle branch February 24, 2026 00:28
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.

3 participants