Skip to content

Keep WebMCP reminders in content flow and preserve footer controls - #239

Merged
chen21019 merged 1 commit into
mainfrom
fix/webmcp-pending-review-layout-20261101
Nov 1, 2026
Merged

chen21019 merged 1 commit into
mainfrom
fix/webmcp-pending-review-layout-20261101

Conversation

@chen21019

Copy link
Copy Markdown

The pending-plan reminder rendered after the authenticated footer could intercept normal clicks on language and CLI controls. Mount the same explicitly gated review component once in authenticated main content; retain its review observer, global modal and manual-consent behavior. Scope wrapping and existing theme tokens to the reminder so long target paths, translated controls, RTL and narrow viewports remain usable.

Validation: the directly affected collaboration entry passed 35 cases once. One new QUnit regression exercises 96 width/direction/theme/copy/path combinations with actual emitted theme/vendor CSS and current scoped Sass in a single local browser. Two-theme normal/hover/keyboard-focus samples pass contrast checks; all three templates parse. These are isolated source/layout results, not the outstanding published/native workflow PASS. The earlier qualified 27ba candidate remains preserved and unpublished; fresh full/enabled producer, CodeQL, original artifact/SBOM/attestation qualification and QA8080 workflows are required. No session/auth/API/dependency changes or production access; versions 1.6.203/bridge0.2.5 remain unpublished and their tags have not been created.

@chen21019
chen21019 requested a review from a team as a code owner November 1, 2026 05:34
@chen21019
chen21019 force-pushed the fix/webmcp-pending-review-layout-20261101 branch from eba611e to d97a899 Compare November 1, 2026 05:39
@chen21019
chen21019 merged commit ae409b8 into main Nov 1, 2026
4 checks passed
@chen21019
chen21019 deleted the fix/webmcp-pending-review-layout-20261101 branch November 1, 2026 05:47
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.

2 participants