Repository navigation
Console polish: reference layout, one card style, daisyUI throughout - #46
Merged
Merged
Conversation
Feedback 1 (rail overflow): status dots are now the status component inside indicator-item, pinned INSIDE the 44px tile bounds via .rail-dot (--indicator-x/y 0, 6px insets) instead of hanging off the corner; the app list scrolls vertically with overflow-x hidden and side padding. Feedback 2 (unified cards): one card style everywhere (card card-border bg-base-100, radius-box hairline, no shadow). --depth: 0 kills daisyUI's built-in shadows; removed every shadow-* class. Feedback 3 (Account section): removed from the app sidebar; Invite codes and Admin stay in the account dropdown only. Feedback 4 (reference gap): light-gray canvas with 8px gaps; white sidebar + main panels (radius 1.5rem, hairline, internal scroll, page never scrolls); rail transparent and flush left (8px 8px 8px 0 inset, 0 left margin); sidebar header (semibold name, muted URL, status badge-soft + status dot, collapse top-right); soft-gray menu-active; neutral-count + soft-Soon badges. Feedback 5 (daisyUI for real): status/indicator/avatar/menu/card/stats/ stat/list/list-row/table/join/tabs-box/btn/drawer/dropdown/tooltip/ skeleton/alert/collapse/badge/kbd/divider/loading all daisyUI; deleted the tints, the stats strip, custom rows. console.css 454 -> 469 lines; what stays custom: theme blocks, shell grid, rail-tile frame + dot inset, charts, log viewer, skip link, boot/empty shells, with reasons in the file header. Feedback 6 (expand left): PanelLeftOpen button at the far left of the main header when collapsed; state persists in localStorage. Feedback 7 (flush rail): no card background, no border, flush left. Feedback 8 (usability): shared EmptyState (icon/title/action, plain variant inside cards); Retry on every error alert; copy buttons swap to check + Copied tooltip; stat values medium weight; legend uses status dots; hamburger opens a drawer on phones; dark theme matches. Verified: Playwright vs real router + seeded absurd data, screenshots in /tmp/outer-console-v4-*.png, no doc h-scroll, no box outside its panel, zero CSP/console errors. cargo fmt/clippy/test green.
…tus mark in the header
Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Owner feedback on the console (8 points) against the reference design (
console-reference-2.jpg).indicator-items translated half outside each tile; now daisyUIstatusinsideindicator-item, inset within the tile; the rail list isoverflow-x: hidden.card card-border bg-base-100everywhere;--depth: 0in both themes; everyshadow-*removed (floating menus keep a hairline border).status-dot legend, pill range picker.statsstrip and custom rows.console.cssstays ~470 lines: theme blocks, the 3-pane shell, chart/log styles and a few daisyUI overrides (soft-gray active row, drawer height/z-index).aria-label+ tooltip; skeletons match the layout; phones use a drawer from a hamburger at the left; dark theme matches.Follow-up commit: the sidebar ran past the viewport bottom (daisyUI's open drawer-side is 100vh inside an 8px-inset shell), section titles uppercase, one status mark in the sidebar header.
Live (
outer:bde9436): Overview at 1440/1024/390, collapsed, Agent actions, dark — rail, sidebar and main all span 8 px to viewport−8 px, no horizontal document scroll, zero console errors. 458 tests, build + validate green.CodeRabbit is out of free reviews; merging on green CI plus the live check.