Skip to content

[Application] Improve fastforward playback cycling#240

Merged
learncold merged 1 commit into
mainfrom
codex/fastforward-cycle
May 14, 2026
Merged

[Application] Improve fastforward playback cycling#240
learncold merged 1 commit into
mainfrom
codex/fastforward-cycle

Conversation

@95x8x9
Copy link
Copy Markdown
Collaborator

@95x8x9 95x8x9 commented May 14, 2026

Summary

  • change the fast forward control to cycle through normal, x2, and x3 playback instead of entering a one-shot fast-forward state
  • update run-status messaging to show x2 fastforward, x3 fastforward, and running based on the active playback mode
  • keep pause/stop/result controls aligned with the new playback-speed behavior

Related Issue

  • None (application-only PR)

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • Not run (reason below)

Not run: ctest --preset test-debug was skipped because this change only affects application-layer playback UI behavior and the app build completed successfully.

Risks / Follow-up

  • this change increases simulation stepping per timer tick for x2 and x3 modes, so we should confirm the perceived speed and status text in the Qt UI manually

@learncold learncold force-pushed the codex/fastforward-cycle branch from 25ff018 to cb736e1 Compare May 14, 2026 05:04
@learncold learncold marked this pull request as ready for review May 14, 2026 05:04
@learncold learncold self-requested a review as a code owner May 14, 2026 05:04
@learncold learncold merged commit b14212f into main May 14, 2026
4 checks passed
@learncold learncold deleted the codex/fastforward-cycle branch May 14, 2026 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants