Yowkees / keyball

keyball is split keyboard has 100% track ball
GNU General Public License v3.0
576 stars 405 forks source link

Disable minor features #533

Closed koron closed 5 months ago

koron commented 6 months ago

Disabled features

Others:

okuRaku commented 6 months ago

突然のコメントで失礼します。

私は自作キーボード自体素人で今日手に入れたばかりのkeyball61キットすら組み立て始めてないのですが、事前準備としてkeymapを勉強していましたところ、KC_GRV (`)の入力方法がどうも見つかりませんでした。さらに探ったら、このPRを見つけました。もしかしてKC_GRVを入力するために、QMKのGRAVE_ESC_ENABLE機能が利用されているでしょうか? その場合、無効化されたら、viaマップでどうやってKC_GRVを入力できるのでしょうか?

私の知識が足りないだけで入力方法ほかにあるでしょうと、思いながらコメントさせていただきました。 申し訳ありません。

koron commented 6 months ago

@okuRaku コメントをありがとうざいます!

GRAVE_ESC_ENABLEKC_GRV とは直接の関係がない機能です。 そのためKeyball 61のデフォルトキーマップに KC_GRV が存在しないのは 独立した問題であると考えられます。

GRAVE_ESC_ENABLEQK_GRAVE_ESCAPE (および QK_GESC )というキーコードを増やす機能です。 このキーコードは単一で入力されると GRAVE ~ に、modifier keys と合わせると ESC として解釈される特殊なものです。

詳しくはこちらをご覧ください。 https://docs.qmk.fm/#/feature_grave_esc

マイナーな機能であるためデフォルトのファームウェアからは消しても良いと考えています。


Thank you for your comment!

GRAVE_ESC_ENABLE feature doesn't relate with KC_GRV keycode directly. Therefore, the absence of KC_GRV in the default keymap of Keyball 61 is considered to be an independent problem.

GRAVE_ESC_ENABLE is a feature that increases key codes QK_GRAVE_ESCAPE and QK_GESC. These keycodes are special keycodes that when typed alone are interpreted as GRAVE ~ and when combined with a modifier key are interpreted as ESC.

See https://docs.qmk.fm/#/feature_grave_esc for detail.

Since it is a minor feature, I think it is okay to remove it from default firmwares.

okuRaku commented 6 months ago

Thank you very much for your reply!

I think my confusion was, looking at the recommended via keymap in this repo I could not find any way to enter the grave (`) key. Therefore, I mistakenly assumed it must rely on this functionality.

If it is not related, may I ask directly, how do we input grave (`) with the recommend via layout?

I apologize for my question based on lack of understanding and experience. Thank you for any further information!

Of course I have no objection to the change in this PR!

okuRaku commented 6 months ago

Sorry again for my question unrelated to the change. I also did not realize it utilized those QK_ keycodes. I think after I have more experience I will understand. Feel free to ignore!! Thank you.

koron commented 6 months ago

@okuRaku Don't hesitate :+1:

Your question is originally caused by the difficulty of understanding QMK, Keyball, and this PR, and in most cases, there is a lack of documentation. I believe that this discussion will increase its documentation and help others in the future.

This is very valuable.