Repository navigation
fix horizontal scrolling in chat - #430
Conversation
|
@codex review Please check transcript overflow containment, long-text wrapping, and independent table/code scrolling across mobile widths and app zoom. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 238ad7602e
ℹ️ 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".
| } | ||
| .zen-moments { | ||
| position: relative; | ||
| overflow-x: hidden; |
There was a problem hiding this comment.
At narrow mobile widths or higher app zoom, this clips the focused-moment indicator because .zen-moment.is-focus::before is positioned 22px to the left of the moment while the moment has only a 4% side margin. For example, in the 320px layout the central column is about 284px wide, leaving an approximately 11px margin, so the cursor is entirely outside .zen-moments and becomes invisible in keyboard browse mode. Contain overflowing message content without clipping this intentional gutter affordance, or reposition the cursor inside the scrollport.
Useful? React with 👍 / 👎.
|
Preview removed. |
|
@codex review Fixed the narrow-screen browse cursor gutter. The 90-case width/zoom matrix now also checks that the cursor remains fully visible and outside the text, while retaining vertical and independent code/table scrolling. |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
This branch was successfully deployed
1 participant
Long URLs, unbroken text and wide tables could make the entire Chat transcript scroll sideways on mobile, leaving ordinary messages clipped after a horizontal swipe.
Keep the transcript vertical, wrap long text, and let wide tables scroll inside their own bounds. Code blocks retain their existing horizontal scrolling.
Validation:
The temporary comparison demo is not part of this change.