Cleanup - #47
Conversation
![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: 26917bec94
ℹ️ 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".
| <div className="w-3 h-3 rounded-full bg-border-muted" /> | ||
| <div className="w-3 h-3 rounded-full bg-border-muted" /> | ||
| <div className="w-3 h-3 rounded-full bg-border-muted" /> | ||
| <div className="hero-terminal mt-8"> |
There was a problem hiding this comment.
The new terminal wrapper no longer has bg-surface-terminal, but the existing terminal palette in index.css is scoped through selectors such as .bg-surface-terminal .text-secondary, .bg-surface-terminal .text-tertiary, and .bg-surface-terminal .text-code-comment. As a result, the terminal preview now falls back to the general theme colors. In the light theme this renders status text such as Solution logged with the dark --secondary color (#006e16) against the dark terminal background instead of the intended bright terminal color. Retain the scoping class or add equivalent .hero-terminal overrides.
Useful? React with 👍 / 👎.
| <div className="flex items-center gap-1.5 sm:gap-2"> | ||
| <ModeToggle /> | ||
| <UserMenu /> | ||
| <MobileNav links={links} /> |
There was a problem hiding this comment.
On screens below sm, this adds a third visible control while the authenticated UserMenu still renders the full, unconstrained session.user.name. The logo, mode toggle, account button, and new burger button therefore overflow the header on narrow phones or for longer user names; the anonymous case avoids this only because its sign-in button is hidden on mobile. Truncate or hide the desktop-style account button at this breakpoint, or expose the account actions through the mobile menu.
Useful? React with 👍 / 👎.
No description provided.