fix: [1078] 譜面明細子画面のショートカットキーの下キーの初期カーソル位置誤りを修正

[cwtickle]

🔨 変更内容 / Details of Changes

1. 譜面明細子画面のショートカットキーの下キーの初期カーソル位置誤りを修正

• 譜面明細子画面のショートカットキーの下キーについて、初期位置に誤りがあり、
  初回のみ下キーを2回押さないと次の明細にならないようになっていました。

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

1. PR improve: [1076] 譜面明細子画面を内部的にページとして分離 #2090 の考慮漏れです。

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

📝 その他コメント / Other Comments

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f9162770-5c23-42b1-a120-31724eff2dce

📥 Commits

Reviewing files that changed from the base of the PR and between e867c8b and 5cf4dd7.

📒 Files selected for processing (1)

• js/danoni_main.js

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Enhanced ArrowDown key navigation in the score detail page. The key now properly cycles through focus targets sequentially instead of consistently returning to the initial item.

Walkthrough

The scoreDetail page's ArrowDown shortcut target was changed from always using the first cursor position to dynamically navigating through available cursor positions using the nextPos() function, enabling cycling through multiple targets instead of a fixed selection.

Changes

Shortcut Navigation Update

Layer / File(s)	Summary
Shortcut Wiring js/danoni_main.js	g_shortcutObj.scoreDetail.ArrowDown.id updated to use nextPos(0, 1, g_settings.scoreDetailCursorsOrg.length) instead of hardcoded g_settings.scoreDetailCursorsOrg[0], enabling dynamic cursor position cycling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• improve: [1076] 譜面明細子画面を内部的にページとして分離 #2090 — Also modifies scoreDetail page shortcut wiring in js/danoni_main.js, specifically updating ArrowDown/KeyQ target assignment with nextPos().

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main fix: correcting the initial cursor position error for the down arrow shortcut key in the score detail subscreens, which matches the changeset modification to the ArrowDown cursor initialization logic.
Description check	✅ Passed	The description is directly related to the changeset, detailing the bug in the score detail subscreens where the down arrow key required two presses initially, and referencing PR #2090 as the source of the oversight being corrected.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch hotfix/1078-score-detail-cursor

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}