Skip to content

fix: use dynamic modifier in debug log messages#28

Merged
OlaProeis merged 1 commit intoOlaProeis:masterfrom
abcd-ca:fix/keyboard-shortcut-hint
Jan 19, 2026
Merged

fix: use dynamic modifier in debug log messages#28
OlaProeis merged 1 commit intoOlaProeis:masterfrom
abcd-ca:fix/keyboard-shortcut-hint

Conversation

@abcd-ca
Copy link
Contributor

@abcd-ca abcd-ca commented Jan 19, 2026

Summary

  • Update debug log messages to use modifier_symbol() instead of hardcoded 'Ctrl' or 'Cmd/Ctrl' strings
  • Ensures consistent platform-appropriate modifier display (Cmd on macOS, Ctrl elsewhere)

Changes

  • src/app.rs: 4 debug messages updated
  • src/ui/view_segment.rs: 1 tooltip updated

Test plan

  • Build with cargo build
  • Run on macOS and verify debug logs show 'Cmd' prefix
  • Run on Linux/Windows and verify debug logs show 'Ctrl' prefix

Update debug log messages to use modifier_symbol() instead of
hardcoded 'Ctrl' or 'Cmd/Ctrl' strings. This ensures consistent
platform-appropriate modifier display (Cmd on macOS, Ctrl elsewhere).
@OlaProeis OlaProeis merged commit b6a4d7f into OlaProeis:master Jan 19, 2026
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.

2 participants