Skip to content

[codex] Fix TV section navigation focus - #17

Draft
AndreKalberer wants to merge 1 commit into
mainfrom
codex/hbr005-tv-focus
Draft

AndreKalberer wants to merge 1 commit into
mainfrom
codex/hbr005-tv-focus

Conversation

@AndreKalberer

Copy link
Copy Markdown
Owner

Switching TV sections could leave focus on the document body after the focused card was removed. The broad [data-section] selector also matched the body, which carries the current section, so focus restoration and navigation state could target it instead of the visible navigation button.

Scope section listeners, active state, and focus restoration to .tv-nav button[data-section]. Add an offline DOM regression capture to the existing TV smoke runner covering Home Continue to LG Back (461), section changes after card removal, My List, the brand link, and return from detail/search overlays. It checks visible navigation focus, exactly one active navigation button, and absence of navigation state on the body.

Validation passed:

  • npm test and npm run test:tv-smoke, including the new focus capture.
  • TV staging and LG packaging; independent review found no findings.
  • Combined 2.3.8 integration: focus capture passed against source, staged LG content, and the extracted LG package. That integration included the separate HLS staging fix in [codex] Fix literal TV staging asset replacement #16; this PR contains only the focus selectors and smoke integration.
  • Staged diff whitespace checks and regression-script syntax check.

Physical LG remote/device verification remains unperformed. No desktop, version, dependency, generated package, or installer changes are included.

This branch has not been deployed

No deployments
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.

1 participant