fix(ui): turn navigator no-op when re-selecting the on-screen turn #57

Merged

developer merged 1 commits from feature/issue-45-turn-navigator-noop into development

2026-05-26 05:13:24 +00:00

Author	SHA1	Message	Date
Ilia Denisov	e82c9f8bbd	fix(ui): no-op when re-selecting the turn already on screen Tests · UI / test (push) Waiting to run Details Tests · UI / test (pull_request) Successful in 3m35s Details Clicking the current-turn row in the header turn navigator while already viewing it routed through returnToCurrent() → viewTurn(currentTurn), which re-fetches the live report and flips the view through `loading`. At turn 0 the only row is the live turn, so the dropdown always fired a pointless backend round-trip and redraw. Guard goToTurn() against re-selecting the on-screen turn (turn === viewedTurn): just close the popover and stop. Leaving history is unaffected — there the viewed turn differs from the target. Closes #45 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-26 00:18:30 +02:00

Author

SHA1

Message

Date

Ilia Denisov

e82c9f8bbd

fix(ui): no-op when re-selecting the turn already on screen

Tests · UI / test (push) Waiting to run

Details

Tests · UI / test (pull_request) Successful in 3m35s

Details

Clicking the current-turn row in the header turn navigator while
already viewing it routed through returnToCurrent() →
viewTurn(currentTurn), which re-fetches the live report and flips the
view through `loading`. At turn 0 the only row is the live turn, so
the dropdown always fired a pointless backend round-trip and redraw.

Guard goToTurn() against re-selecting the on-screen turn
(turn === viewedTurn): just close the popover and stop. Leaving
history is unaffected — there the viewed turn differs from the target.

Closes #45

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

2026-05-26 00:18:30 +02:00

fix(ui): turn navigator no-op when re-selecting the on-screen turn #57

1 Commits