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

feat: [0777] keyGroupOrderをカスタムキーの1項目として追加 他 #1612

Merged
merged 1 commit into from Feb 3, 2024

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Feb 3, 2024

🔨 変更内容 / Details of Changes

  1. keyGroupOrder(キーグループの表示制御)をカスタムキーの1項目として追加しました。
    譜面ヘッダー側と併用した場合は、譜面ヘッダーの方が優先されます。

<利用例>

  • デフォルトのキーグループが0に割り当てされることを利用して、
    部分キーグループを1~4に割り当て、キーコンフィグ上は1~4のグループのみを表示する例です。
|keyCtrl25=D2,D3,D4,D7,D8,D9,W,E,R,U,I,O,7_0,Z,X,C,M,Comma,Period|
|chara25=aleft,aleftdia,adown,aup,arightdia,aright,bleft,bleftdia,bdown,bup,brightdia,bright,7_0,cleft,cleftdia,cdown,cup,crightdia,cright|
|color25=4,4,4,4,4,4,3,3,3,3,3,3,0,0,0,2,0,0,0,1,1,1,1,1,1|
|stepRtn25=0,-45,-90,90,135,180,0,-45,-90,90,135,180,7_0_0,0,-45,-90,90,135,180|
|pos25=0,1,2,4,5,6,0.01,1.01,2.01,4.01,5.01,6.01,0.02,1.02,2.02,3.02,4.02,5.02,6.02,0.03,1.03,2.03,4.03,5.03,6.03|
|keyGroup25=0/1,0/1,0/1,0/1,0/1,0/1,0/2,0/2,0/2,0/2,0/2,0/2,0/3,0/3,0/3,0/1/2/3/4,0/3,0/3,0/3,0/4,0/4,0/4,0/4,0/4,0/4|
|keyGroupOrder25=1,2,3,4|
  1. 複数のキーパターンでキーグループ数が異なる場合にエラーで止まることがある問題を修正しました。

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

  1. トランスキー以外でkeyGroupOrderを使う可能性が考えられるため。
  2. 複数パターンがあり、片方のキーグループが存在しないグループを選択している場合にエラーが発生していました。ただ、通常は発生しえない状況だと思います。

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

  • 譜面ヘッダーのkeyGroupOrderはパターン毎に定義できないため、過去バージョンに2.の修正を反映する必要はありません。今回の変更用の対応です。
  • なお、ステップゾーンは従来通り重ねる実装になっているため、空押しした場合には一番上に表示されているステップゾーンのみが光ります。あくまでキーコン上の見た目を解消する変更です。
    上記の例では、posXを0.01ずつずらすことでうっすら見えるようにしています。

@cwtickle cwtickle added bug Something isn't working func:KeyConfig need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け labels Feb 3, 2024
Copy link

codeclimate bot commented Feb 3, 2024

Code Climate has analyzed commit f12ca8e and detected 0 issues on this pull request.

View more on Code Climate.

@cwtickle cwtickle removed bug Something isn't working need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け labels Feb 3, 2024
@cwtickle cwtickle merged commit a47a3a9 into develop Feb 3, 2024
4 checks passed
@cwtickle cwtickle deleted the feature/0777-keyGroupOrder branch February 8, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant