refactor: simplify quick start OS selector#107
refactor: simplify quick start OS selector#107thewilloftheshadow merged 1 commit intoopenclaw:mainfrom
Conversation
|
@geoffrey-xiao is attempting to deploy a commit to the Jamie's projects Team on Vercel. A member of the Team first needs to authorize it. |
geoffrey-xiao
changed the title
![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: 0f9fc8e85f
ℹ️ 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".
| const showBetaControls = currentMode === 'oneliner' || currentMode === 'quick'; | ||
| if (osIndicator) osIndicator.style.display = showOsControls && !osPickerExpanded ? 'flex' : 'none'; | ||
| if (osSwitch) osSwitch.style.display = showOsControls && osPickerExpanded ? 'flex' : 'none'; | ||
| if (osSwitch) osSwitch.style.display = showOsControls ? 'flex' : 'none'; |
There was a problem hiding this comment.
Showing the full os-switch whenever one-liner mode is active makes the Quick Start header wider than before, and on mid-size mobile/tablet widths (roughly 481–560px) it can overflow because .code-header only wraps at @media (max-width: 480px). In this range, the mode tabs + OS toggle + beta toggle no longer reliably fit on one row, so controls can clip or overflow horizontally; this regression is introduced by always rendering the larger OS control instead of the compact indicator.
Useful? React with 👍 / 👎.
2 participants
Description
This simplifies the Quick Start OS selector by removing the extra
changebutton and showing themacOS/Linux/Windowstoggle directly in one-liner mode.Why
The previous UI split a simple binary choice across two controls, which added friction and made the interaction less obvious. Showing the OS toggle directly is clearer and faster.
Testing
src/pages/index.astrobun run buildsuccessfullybefore
after
