Conversation
Replace tab-based navigation with a single active session flow in the app shell and sidebar. Remove the persisted ui_state save/load path from the frontend and Tauri backend, keeping only the unrelated home-directory command in a dedicated system command module. Update affected tests and fixtures to match the new session-only model.
![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: 37fff37f70
ℹ️ 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".
| onHomeClick={() => handleNavigate("home")} | ||
| /> | ||
|
|
||
| <div className="flex flex-1 min-h-0 overflow-hidden"> |
There was a problem hiding this comment.
The new layout renders the app body starting at the very top (AppShell now goes straight into the main flex row), but the Tauri window is still configured with an overlay/hidden title bar and macOS traffic lights (src-tauri/tauri.conf.json), so there is no longer a dedicated top gutter/offset to keep native controls clear of app UI. In practice on macOS this causes the traffic-light controls to overlap the sidebar header area and reduces reliable drag space; please add a replacement top drag/offset region (equivalent to the previous pl-20 handling) when the tab bar is removed.
Useful? React with 👍 / 👎.
Merge origin/main (tabs removal refactor block#70) and fix two code review items: - Register @tailwindcss/typography plugin via @plugin directive for prose classes - Fix accordion keyframes to use --radix-accordion-content-height (not --bits-) - Remove GooseIcon home button from sidebar header (removed upstream) - Adopt tab→session rename throughout sidebar components Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 participant
Summary