Skip to content

[fix] 校舎環境に合わせて+-キーをP, Mに変更#33

Merged
k1n4mur4 merged 2 commits intomainfrom
release/submission-1
Feb 23, 2026
Merged

[fix] 校舎環境に合わせて+-キーをP, Mに変更#33
k1n4mur4 merged 2 commits intomainfrom
release/submission-1

Conversation

@gaotoast
Copy link
Owner

概要

校舎環境に合わせて+-キーをP, Mに変更

関連issue

なし

変更点

  • キーコードの定義にP, Mを追加
  • +, -キーを使用していた部分をP, Mに変更

懸念点

なし

レビュー観点

  • 変更箇所に漏れがないか

その他

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 pull request modifies the key bindings for adjusting FOV, brightness, and diameter from the + and - keys to P and M keys to accommodate the school environment's keyboard layout.

Changes:

  • Added KEY_P and KEY_M constant definitions for both macOS and Linux platforms
  • Updated key handler logic to use KEY_P and KEY_M instead of KEY_PLUS and KEY_MINUS
  • Updated all user-facing help text to display the new keybindings (P/M instead of +/-)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
includes/config.h Added KEY_P and KEY_M definitions for both macOS and Linux platforms
srcs/key_handler.c Updated get_adjust_params function to check for KEY_P and KEY_M instead of KEY_PLUS and KEY_MINUS
srcs/put_info.c Updated help text strings to display "P (Increase), M (Decrease)" instead of "+ (Increase), - (Decrease)"

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

Copy link
Collaborator

@k1n4mur4 k1n4mur4 left a comment

Choose a reason for hiding this comment

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

いいです。

@k1n4mur4 k1n4mur4 merged commit d3a5104 into main Feb 23, 2026
2 checks passed
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