Skip to content

Focus Goal composer after it expands - #6282

Closed
LIHUA919 wants to merge 2 commits into
loopx-project:mainfrom
LIHUA919:codex/goal-showcase-focus-repair
Closed

LIHUA919 wants to merge 2 commits into
loopx-project:mainfrom
LIHUA919:codex/goal-showcase-focus-repair

Conversation

@LIHUA919

@LIHUA919 LIHUA919 commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Clicking the folded composer on a Goal Overview expands the textarea but can leave keyboard focus on the button: its animation-frame callback may run before React mounts the textarea. The packaged goal-showcase journey reproduced this on main b760c1eb3. Focus now moves in the component's layout effect after the open state commits, so one click lands in the editor. The change reuses the existing composer state and ref and does not change layout or authority behavior.

Head 2202518cb integrates latest main 36a2f7106, including the merged request-scoped refresh feedback repair from PR #6273. The PR diff against main remains one component file. On this head, the dashboard and packaged Chat bundle built, the packaged goal-showcase focus journey passed, and packaged blocked-reason-readback passed with the upstream held-response regression. Temporary local dependencies were removed after validation; the worktree is clean.

The prior head's Release Artifacts failure was in the shared refresh path, and its Python browser failure was caused by CI jobs that did not install their Python source checkout. PR #6287 proposes the latter CI-only repair and is still under review. Fresh exact-head remote CI and maintainer review are required before merge; the local browser passes do not qualify those remote checks.

Signed-off-by: Lihua <1017343802@qq.com>
Signed-off-by: Lihua <1017343802@qq.com>
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