Skip to content

Commit 43118e4

Browse files
committed
Record CI and phone validation in the spec
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AjDKX9vvNheXYBzjZxnw4W
1 parent cdaaca9 commit 43118e4

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

‎docs/specs/2026-09-29-active-line-stays-visible-on-pause.md‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,8 @@
22

33
## Status
44

5-
Implemented. Unit tests and the build pass locally. The new managed-device test runs in CI. The
6-
owner's phone check is pending. The owner asked for a PR; that does not authorize merging or
7-
publishing.
5+
Validated. Local checks and CI pass, including the new managed-device test. On 2026-09-29 the
6+
owner confirmed on their phone that it works and asked to merge.
87

98
## Context / problem
109

@@ -60,10 +59,10 @@ panel got shorter. Rows growing in place moved nothing.
6059
- [x] `revealScrollDelta` returns 0 for a fitting row, the clipped amount at the bottom, the top
6160
offset at the top, and top-alignment for a row taller than the panel
6261
(`PlaybackArchitectureTest.activeRowRevealMovesOnlyAsFarAsTheClippedPart`).
63-
- [ ] With translations shown only when paused and the spoken line as the last row that fits,
62+
- [x] With translations shown only when paused and the spoken line as the last row that fits,
6463
pausing leaves that line and its translation inside the panel
6564
(`SubtitleUiTest.pausingKeepsTheActiveLineAndItsTranslationOnScreen`, managed device in CI).
66-
- [ ] Owner's phone: the report's scenario no longer needs a scroll.
65+
- [x] Owner's phone: the report's scenario no longer needs a scroll.
6766

6867
## Validation plan
6968

@@ -99,6 +98,6 @@ disappears; a row that was only partly visible keeps the old jump-to-top rule.
9998
`./gradlew formatCheck complexityCheck testDebugUnitTest lintDebug assembleDebug assembleDebugAndroidTest`.
10099
`PlaybackArchitectureTest` ran 22 tests with 0 failures, including the new one.
101100
- Not run locally: `pixel2Api36DebugAndroidTest`, because this host has no KVM. CI's
102-
`managed-device-tests` job runs the new `SubtitleUiTest` case.
103-
- Not run: phone and live YouTube. The owner can check this on the PR's preview APK.
101+
`managed-device-tests` passed on `cdaaca9`, including the new `SubtitleUiTest` case.
102+
- Passed on the owner's phone with live YouTube, reported by the owner on 2026-09-29.
104103
- Final-head CI status is on the PR.

0 commit comments

Comments
 (0)