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

fix: [0680] 一部キーでスクロールの折り返しが外れてしまう問題を修正 #1458

Merged
merged 2 commits into from Apr 5, 2023

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Apr 4, 2023

🔨 変更内容 / Details of Changes

  1. 上下スクロールがあるキーで、キーパターン違いがあるときに
    ステップゾーンの配置が想定と異なる問題を修正しました。

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

  1. PR improve: [0679] カスタムキーの必須項目をkeyCtrlXのみに変更 他 #1456 に対する考慮漏れ。
  • setKeyDfValについて、keyCtrlX_Yが存在するものに対して初期化していましたが、
    これにスクロールの折り返しを管理するdivX_Yの初期化が含まれていました。
  • keyCtrlX_Yが定義されているパターン(X, Y)の中にはdivX_Yがいないパターン(X, Y)があり、後で別パターンからコピーする仕様になっていましたがsetKeyDfValの時点で初期化されてしまうため、divX_Yが存在する状態となってしまい、コピーができませんでした。
    ※別パターンからのコピーは、divX_Yが未定義の場合のみ適用される
  • この部分だけは、charaX_Yが存在するものを基準に初期化する必要がありました。

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

📝 その他コメント / Other Comments

  • すでにリリース済みのバージョンには影響しません。

@cwtickle cwtickle added bug Something isn't working func:KeyConfig labels Apr 4, 2023
@codeclimate
Copy link

codeclimate bot commented Apr 5, 2023

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

View more on Code Climate.

@cwtickle cwtickle merged commit 68db2f0 into develop Apr 5, 2023
4 checks passed
@cwtickle cwtickle deleted the hotfix/0680-customdf branch April 7, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working func:KeyConfig
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant