🤖 perf: improve small-screen UX and reduce mobile CPU spikes#1778
🤖 perf: improve small-screen UX and reduce mobile CPU spikes#1778ibetitsmike wants to merge 4 commits intomainfrom
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8a02d96781
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
@codex review Handled the small-screen terminal popout path with a best-effort |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 460ba760c7
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
src/browser/components/RightSidebar/FileViewer/TextFileViewer.tsx
Outdated
Show resolved
Hide resolved
|
@codex review Updated |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 59eb44e2ec
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
src/browser/components/RightSidebar/FileViewer/TextFileViewer.tsx
Outdated
Show resolved
Hide resolved
|
@codex review Scoped the large-file “render highlighted” override by |
|
Codex Review: Didn't find any major issues. Bravo. ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
c29a910 to
6bdb3b6
Compare
What
Mobile + small-screen improvements:
.safe-area-top-bleedutility and apply it to top bars so the header background extends into the notch area without clipping interactive content.<RightSidebar />.ChatPaneby computing group info in one pass.TextFileViewer(render raw<pre>with an explicit opt-in to full render).Why
On mobile / small screens the right sidebar is not usable and large chats/files can cause CPU spikes and jank. These changes prioritize:
Validation
make static-checkGenerated with
mux• Model:openai:gpt-5.2• Thinking:high• Cost:$8.32