Skip to content

Fix episode context menu focus#25

Merged
rovkinmax merged 1 commit into
masterfrom
PUB-2
Jul 7, 2026
Merged

Fix episode context menu focus#25
rovkinmax merged 1 commit into
masterfrom
PUB-2

Conversation

@rovkinmax

Copy link
Copy Markdown
Owner

Task Summary

  • Fixes focus handoff for episode/season context menus in details and player episode panels.
  • Keeps EpisodesPanel focus containment by default, and only allows focus to leave the panel while an episode context menu is open.
  • Updated both EpisodesPanel call sites found by search: details seasons list and player settings episodes list.

Compliance Review

Passed. Rule sources checked: root AGENTS.md, .kent/commands/compliance-review.md, .kent/project-contract.md, PUB-2 task body, and .todo/bugfix-season-context-menu-focus/diagnosis.md. Reviewed diff contains only:

  • app/src/main/java/com/kino/puber/ui/feature/player/component/EpisodesPanel.kt
  • app/src/main/java/com/kino/puber/ui/feature/details/component/DetailsScreenContent.kt
  • app/src/main/java/com/kino/puber/ui/feature/player/component/PlayerSettingsPanels.kt

No compliance findings were reported.

Verification

  • ./tools/agentw :app:compileDevDebugKotlin - passed (BUILD SUCCESSFUL, configuration cache reused).
  • Post-fix UI evidence reviewed in .todo/bugfix-season-context-menu-focus/: menu focus lands on the context-menu action, DPAD down moves within the menu, dismissal returns focus to the episode card, and normal no-menu left containment remains active.
  • rg "EpisodesPanel\\(" app/src/main/java/com/kino/puber/ui/feature confirmed both production call sites were updated.

Skipped Checks / Blockers

  • This PR shipping step did not rerun build or device smoke checks; it used the already completed implementation, verification, and mandatory Compliance Review evidence.
  • .kent/commands/ship-pr.md was missing in this PUB-2 worktree. For shipping procedure, I followed the project contract and the equivalent command file present in sibling Puber worktrees.

@rovkinmax rovkinmax merged commit 029f628 into master Jul 7, 2026
3 checks passed
@rovkinmax rovkinmax deleted the PUB-2 branch July 7, 2026 07:25
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