Skip to content

Conversation

@ginsarix
Copy link

@ginsarix ginsarix commented Jan 7, 2026

This PR fixes two inconsistencies in the keyboard key descriptions:

  1. MetaRight missing "Right ⌘": While MetaLeft correctly includes "Left ⌘" in its description, MetaRight was missing the corresponding "Right ⌘" label.

  2. ContextMenu incorrectly labeled: The ContextMenu key currently shows "Right ⌘", but this doesn't match the physical layout of Mac keyboards

These changes improve accuracy and maintain consistency across the keyboard layout descriptions.

How to test

  • Run the development server
  • Open the app in a browser
  • Verify MetaRight now includes "Right ⌘" in its description card in the UI when pressed and in its description column in the table
  • Verify ContextMenu no longer includes "Right ⌘" in its description card in the UI when pressed and in its description column in the table

Acceptance Criteria

  • MetaRight description includes "Right ⌘"
  • ContextMenu description does not include "Right ⌘"

Pre-merge checklist

  • The PR relates to a single subject with a clear title and description in grammatically correct, complete sentences.
  • Verify that feature branch is up-to-date with main (if not - rebase it).
  • Double check the quality of commit messages.

Related Issues

Fixes #345

- Add "Right ⌘" to MetaRight to match MetaLeft consistency.
- Remove "Right ⌘" from ContextMenu as it is incorrect for Mac layouts.
@ginsarix ginsarix requested a review from a team as a code owner January 7, 2026 11:18
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.

MetaRight description missing "Right ⌘"

1 participant