Skip to content

Fix footer browser selectors after model pill grouping - #749

Merged
witbrock merged 1 commit into
mainfrom
codex/von-0343d65affea9cfe
Sep 17, 2026
Merged

witbrock merged 1 commit into
mainfrom
codex/von-0343d65affea9cfe

Conversation

@witbrock

Copy link
Copy Markdown
Member

PR #748 put the reasoning button inside the model pill, so the mobile-footer and native-zoom browser checks now match two buttons and fail Playwright's strict selector check when opening model settings. Select the pill's direct model button in all three affected clicks.

Validation: Chromium with the actual merged footer renderer/CSS and mocked APIs reproduces the old two-match failure, verifies that the replacement selects only the model button, and confirms both model and reasoning controls open settings. All 37 footer execution-status/preferences tests pass; both changed scripts pass syntax checks. Full authenticated mobile and native-zoom campaigns were not rerun for this selector-only repair. No deployment requested.

Manager review follow-up: #V#task_agent_942475d9a53e868bf06a39d637967ed1. No product-code changes.

@witbrock
witbrock merged commit 55c13f8 into main Sep 17, 2026
2 checks passed
@witbrock
witbrock deleted the codex/von-0343d65affea9cfe branch September 17, 2026 22:06
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