Skip to content

Fix mobile chat layout/scrolling and transcript modal UX#89

Closed
rushcreek wants to merge 672 commits intogrp06:mainfrom
rushcreek:fix/mobile-chat-layout-scroll-modal
Closed

Fix mobile chat layout/scrolling and transcript modal UX#89
rushcreek wants to merge 672 commits intogrp06:mainfrom
rushcreek:fix/mobile-chat-layout-scroll-modal

Conversation

@rushcreek
Copy link

Summary

  • fix mobile chat transcript sizing so scrolling works in normal view
  • add a read-only mobile transcript modal with explicit close action
  • keep transcript expand control mobile-only and hide it on desktop
  • fix markdown overflow edge cases for <pre> and inline <code> content
  • avoid restoring Fleet filter to approvals by default on bootstrap
  • add regression tests for transcript modal controls and markdown pre-wrap behavior

Validation

  • npm run typecheck
  • targeted unit tests for:
    • tests/unit/agentChatPanel-controls.test.ts
    • tests/unit/markdownPreWrapRegression.test.ts
    • tests/unit/studioBootstrapWorkflow.test.ts

grp06 added 30 commits February 12, 2026 10:30
grp06 and others added 20 commits February 27, 2026 14:52
@rushcreek
Copy link
Author

Quick UX summary of what this fixes (mobile + chat stability):

Chat scrolling fixed in both normal and expanded transcript views.
Added read-only transcript modal on mobile with a clear Close action.
Added a small expand icon for discoverability, and ensured it’s hidden on desktop.
Fixed markdown overflow regressions caused by long

 and inline  content that could stretch chat width and hide controls.
Prevented Fleet from unexpectedly restoring the Approvals filter by default.
Added regression coverage for:
transcript modal controls
markdown pre-wrap safeguards
focused filter restore behavior
This should stabilize the mobile chat experience and prevent the same layout regressions from reappearing.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 21f8b7e88b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@grp06
Copy link
Owner

grp06 commented Mar 10, 2026

fixed in the latest commit. So much changed it was impossible to merge. I cherry picked your work, thanks for contributing! I'll make sure to merge PRs in a more timely manner next time so your contributions count!

@grp06 grp06 closed this Mar 10, 2026
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