Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Describe key combinations consistently #8577

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

richcarl
Copy link
Contributor

Ensure that the UI is consistent (help messages, menus, etc.) and that
it's easy to grep in the code for key combination mentions.

  • Use + to write key combinations in source code and UI
  • Use Titlecase for modifiers, Uppercase for letter keys.
  • Leave docs unchanged; they consistently use - instead.

richcarl added 2 commits June 13, 2024 14:27
Control always written as Ctrl. Letter keys always uppercase.
Modifiers (Shift, Alt, Ctrl) always titlecase.
Ensure that the UI is consistent (help messages, menus, etc.) and that
it's easy to grep in the code. Note that the documentation has a
different style and uses `-` to describe key combinations.
Copy link
Contributor

github-actions bot commented Jun 13, 2024

CT Test Results

    11 files     400 suites   3h 32m 54s ⏱️
 6 326 tests  5 918 ✅   401 💤 7 ❌
11 955 runs  10 709 ✅ 1 239 💤 7 ❌

For more details on these failures, see this check.

Results for commit ea381d0.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Jun 17, 2024
@jhogberg jhogberg self-assigned this Jun 24, 2024
@jhogberg jhogberg added the testing currently being tested, tag is used by OTP internal CI label Jul 17, 2024
@jhogberg jhogberg merged commit c70a8a2 into erlang:master Jul 29, 2024
23 of 26 checks passed
@jhogberg
Copy link
Contributor

Merged! Thanks for the PR, and sorry about the wait -- I totally forgot about this :-(

@richcarl richcarl deleted the consistent-key-combos branch August 3, 2024 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants