Skip to content

refactor(shell): replace icon rail with status bar#4

Merged
rivodotlove merged 1 commit into
mainfrom
refactor/status-bar
May 29, 2026
Merged

refactor(shell): replace icon rail with status bar#4
rivodotlove merged 1 commit into
mainfrom
refactor/status-bar

Conversation

@rivodotlove

@rivodotlove rivodotlove commented May 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Drop 56px activity rail. Top items (tools/search/favourites/open-tabs) were unimplemented stubs.
  • Move shortcuts/info/settings to a 24px VS Code-style status bar pinned bottom-right.
  • Frees horizontal space for sidebar + content panes.

Test plan

  • pnpm dev — verify status bar renders bottom-right, 24px tall
  • Sidebar resize still works
  • No icon rail column on left edge
  • Status bar buttons show tooltips (Shortcuts/Info/Settings)
  • pnpm check clean

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Added a status bar at the bottom of the app displaying icon controls.
  • Refactor

    • Reorganized the app shell layout with a resizable sidebar panel.

Review Change Stack

Drop the 56px activity rail. Top items (tools/search/favourites/
open-tabs) were unimplemented stubs. Move shortcuts/info/settings
to a 24px VS Code-style status bar pinned bottom-right, freeing
horizontal space for the sidebar and content panes.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 29, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8c113490-3dc2-4032-b164-bdc3be7c3e97

📥 Commits

Reviewing files that changed from the base of the PR and between b130bd6 and fc01797.

📒 Files selected for processing (1)
  • src/components/shell/app-shell.tsx

📝 Walkthrough

Walkthrough

The shell component is refactored to replace a left icon rail with a horizontal resizable sidebar layout. A new bottom status bar component is added with status bar item definitions. Lucide-react imports are updated to include only the icons required by the status bar.

Changes

Shell Layout and Status Bar Refactoring

Layer / File(s) Summary
Layout refactoring and status bar UI
src/components/shell/app-shell.tsx
Lucide-react imports are updated to support the new status bar icons. The AppShell layout is refactored to replace the vertical icon rail with a horizontal resizable sidebar. A new STATUS_BAR_ITEMS constant and StatusBar() component are introduced to render icon buttons in a fixed-height row at the bottom of the shell.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The shell grows sleeker, sidebar slides in wide,
A status bar hops to the bottom with pride,
Icons dance softly in rows,
Layout refactored—the new design glows!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/status-bar

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


Comment @coderabbitai help to get the list of available commands and usage tips.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
devbox fc01797 Commit Preview URL

Branch Preview URL
May 29 2026, 05:34 PM

@rivodotlove rivodotlove marked this pull request as ready for review May 29, 2026 17:34
@rivodotlove rivodotlove merged commit 8dc9869 into main May 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant