[UtkarshUsername]

Problem

On the new thread page, pressing the permissions label below the composer briefly shrinks it, making the text appear to jump as the permissions menu opens.

Change

Mark the permissions button with aria-haspopup="menu". This describes the menu it opens and excludes it from ComposerControl's active press scale, matching the other popup triggers. The change applies to web and desktop, which share this component.

Scope and approval

This is a small, focused fix for an obvious visual bug in an existing menu trigger. It adds the missing popup semantics without changing permission behavior.

Verification

Before:

permissions-before.mov

After:

permissions-after.mov

• vp run check:changed upstream/main passed formatting, lint, both knip checks, and affected typechecks.
• Source inspection confirms the active scale selector excludes buttons with aria-haspopup.

Model: gpt-6.1-sol. Harness: Codex in T3 Code.

[macroscopeapp]

Approvability

Verdict: Approved at 5d77b21

Macroscope's review found this PR approvable — Adds menu-popup semantics to the permissions button, which also excludes it from the existing press-shrink animation. Permission behavior and product defaults remain unchanged.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 5211acf
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 72768bc2-b1a3-494a-aa73-9e2e9fdf2d55

Commits

Reviewing files that changed from the base of the PR and between 5211acf and 5d77b21.

Files selected for processing: 1

• apps/web/src/components/ComposerContextPermissions.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

Walkthrough

The permissions control now sets aria-haspopup="menu".

Changes: Permissions menu semantics

Layer / File(s)	Summary
Declare menu behavior apps/web/src/components/ComposerContextPermissions.tsx	The permissions control now declares that it opens a menu using aria-haspopup="menu".

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Possibly related PRs

• pingdotgg/t3code#25869: Adds the ComposerControl selector that excludes controls with aria-haspopup; this change adds that attribute to the permissions control.

Suggested reviewers: t3dotgg

Merge Risk: Minimal · up to 5d77b

The change adds menu semantics to the permissions control without an identified behavior or permission regression.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely describes the primary visual fix: preventing the permissions label from shrinking when clicked.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The description includes the required Problem, Change, Scope and approval, and Verification sections. It explains the visual bug, identifies the fix and scope, provides before and after evidence, and …

Full details: Description check

Explanation

The description includes the required Problem, Change, Scope and approval, and Verification sections. It explains the visual bug, identifies the fix and scope, provides before and after evidence, and lists validation results. It does not explicitly state that browser verification was not run, but the description is otherwise complete.

Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}