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: [0574] 重複するid名の解消 #1300

Merged
merged 1 commit into from Aug 1, 2022
Merged

Conversation

cwtickle
Copy link
Owner

@cwtickle cwtickle commented Aug 1, 2022

🔨 変更内容 / Details of Changes

  1. 譜面明細画面周りでID名が重複していた問題を修正しました。
    (共通処理のため、関連するID名も一部変更があります)
  2. g_lblNameObjの設定により、譜面明細画面のID名に英数字以外が入りうる問題も合わせて修正しました。
No 該当箇所 元ID名 変更後ID名
1 譜面明細 - Speedボタンのショートカット scSpeed scSpeedG
2 譜面明細 - Densityボタンのショートカット scDensity scDensityG
3 譜面明細 - ToolDifボタンのショートカット scToolDif scToolDifG
4 譜面明細 - Speed > 全体速度変化回数(Speed) lblSpeed / dataSpeed lblSpeedS / dataSpeedS
5 譜面明細 - Speed > 個別速度変化回数(Boost) lblBoost / dataBoost lblBoostS / dataBoostS

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

  1. デザイン全般を変える際に支障となるため。
  2. updateScoreDetailLabel関数において、2番目の引数はラベル名の元情報となっているが、
    g_lblNameObjを指定していたため、英数字以外のID名となる可能性があった。
    (ID名の項目は5番目の引数があるため、本来はそれを使うべき)

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

📝 その他コメント / Other Comments

  • No.1~3はver27.8.0起因、No.4~5は従来から発生していました。
  • 項番2にあるID名に英数以外が入りうるのは問題のため、過去バージョンにも後で適用する方向で考えます。
    (取り急ぎは現バージョンのみ)

@cwtickle cwtickle added bug Something isn't working func:Settings labels Aug 1, 2022
@codeclimate
Copy link

codeclimate bot commented Aug 1, 2022

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

View more on Code Climate.

@cwtickle cwtickle merged commit d68347d into develop Aug 1, 2022
@cwtickle cwtickle added the need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け label Aug 1, 2022
@cwtickle cwtickle deleted the hotfix/0574-dupli-id branch August 21, 2022 00:08
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:Settings need-to-fix-supportVer. サポートバージョンへの修正が必要な場合にラベル付け
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant