Skip to content

fix(web): show context meter in compact composer - #9430

Merged
maria-rcks merged 2 commits into
pingdotgg:mainfrom
GuilhermeVieiraDev:fix/compact-composer-context-meter
Sep 4, 2026
Merged

fix(web): show context meter in compact composer#9430
maria-rcks merged 2 commits into
pingdotgg:mainfrom
GuilhermeVieiraDev:fix/compact-composer-context-meter

Conversation

@GuilhermeVieiraDev

@GuilhermeVieiraDev GuilhermeVieiraDev commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

What Changed

  • Show the enabled legacy context-window meter in the compact/resting composer.
  • Reserve prompt space for the extra action while keeping the existing 8px action spacing.

Why

The compact composer hid the context-window meter even when the user explicitly enabled the legacy option.

UI Changes

Before

Compact composer without the enabled context meter

After

Compact composer showing the enabled context meter

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for the UI change
  • No animation or interaction change requires a video

Built with GPT-5.6-Sol using the Codex harness in T3 Code.


Note

Low Risk
Localized composer layout and conditional UI in ChatComposer with no auth, data, or API changes.

Overview
When users enable the legacy context-window meter, it now appears in the compact/resting composer instead of being suppressed whenever the composer is collapsed.

The showSecondaryStatus gate that hid ContextWindowMeter in resting mode is removed; visibility follows the same activeContextWindow + contextWindowMeterEnabled rules as the expanded footer. The resting prompt row gets pr-28 when the meter is shown so the single-line editor does not overlap the meter and send/attach controls.

Reviewed by Cursor Bugbot for commit d60de33. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 3, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 579a878

Macroscope's review found this PR approvable — This one-file UI fix restores the existing context meter for users who explicitly enabled it in the compact composer and reserves the necessary layout space. The feature remains off by default, with no schema, security, infrastructure, or static-analysis changes.

You can add or adjust custom eligibility rules. Learn more.

@Mnigos Mnigos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wanted this myself, the resting composer is exactly when I check how close I am to compaction. Code checks out: the meter was already gated on the setting, this just stops hiding it when resting.

@maria-rcks
maria-rcks merged commit f559fe0 into pingdotgg:main Sep 4, 2026
24 checks passed
sheehanmunim added a commit to munimtechnologies/mtcode that referenced this pull request Sep 4, 2026
…tlebar, auth refresh, wordmark)

Brings the fork up to upstream/main f559fe0. Small fixes batch: desktop
panel titlebar interactions (pingdotgg#9591), Connect auth refresh without disconnect
(pingdotgg#9582), sidebar wordmark baseline alignment (pingdotgg#9578), right panel transitions
(pingdotgg#9554), settings sidebar rendering (pingdotgg#9562, pingdotgg#9563), codex restart continuity
(pingdotgg#9560), contributor vouching (pingdotgg#9557), usage proxy dedup (pingdotgg#9584), mobile
markdown images (pingdotgg#8769), antigravity subagent batches (pingdotgg#9579), shell label
syntax (pingdotgg#9371), context meter in compact composer (pingdotgg#9430), disconnected-send
toasts (pingdotgg#9592).

Resolutions (keep-both unless noted):
- ChatComposer: fork voice-session button + showSecondaryStatus kept.
- SettingsSidebarNav: fork MT Teams badge + avatar box kept; upstream Suspense
  wrappers adopted.
- SidebarChrome: fork BrandWordmark kept inside upstream's baseline-aligned
  wordmark row.
- supervisor: upstream fiber-based establishment + replacement loop win (fork
  branches superseded by the auth-refresh rework).

Fork guard script OK.
msegec added a commit to msegec/t3code_rookie that referenced this pull request Sep 4, 2026
Default the meter on while preserving the user opt-out. Include upstream pingdotgg#9430 because nightly 1275 predates its compact-composer fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants