Skip to content

fix(web): keep mobile session header compact#1082

Merged
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/mobile-session-header-reasoning
Jul 19, 2026
Merged

fix(web): keep mobile session header compact#1082
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/mobile-session-header-reasoning

Conversation

@techotaku39

Copy link
Copy Markdown
Contributor

Summary

  • Hide the reasoning ... label in SessionHeader below the sm breakpoint.
  • Keep the label visible in the desktop header.
  • Keep the composer StatusBar reasoning label unchanged on every viewport.

Problem

The reasoning label added in #1016 can wrap onto a third header line on narrow mobile screens. This makes the session header noticeably taller, while the same reasoning value is already visible in the composer status bar.

Using the existing Tailwind sm breakpoint keeps the mobile header to its intended title + metadata layout without removing the desktop status information introduced by #1016.

Test plan

  • bun typecheck
  • bun run test:web (158 files, 1279 tests)
  • Verify on a narrow mobile viewport that SessionHeader no longer shows reasoning ... or gains a third line.
  • Verify the composer StatusBar still shows the active reasoning effort on mobile.
  • Verify SessionHeader still shows the reasoning effort at desktop widths (sm and above).
  • Smoke-test the production Web/Hub build on ports 3016 and 3006.

Related

AI disclosure

This change was prepared with OpenAI Codex (gpt-5.6-sol).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Findings
No findings.

Summary
Review mode: initial
No high-confidence issues found in the one-line web diff. Residual risk: I did not run tests or viewport checks in automation; PR body reports bun typecheck, bun run test:web, and manual mobile/desktop checks.

Testing
Not run (automation).

HAPI Bot

@tiann
tiann merged commit fdf1342 into tiann:main Jul 19, 2026
2 checks passed
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