Repository navigation
improve(web): dashboard design pass with one surface, card and button system - #4259
Open
richiemcilroy wants to merge 6 commits into
Open
richiemcilroy wants to merge 6 commits into
richiemcilroy wants to merge 6 commits into
Conversation
…s onto white surfaces
…ebar and slimmer top bar
… spaces, import and analytics
|
Sandchest reviewed Sandchest reviews every change for vulnerabilities an attacker could use · 1 credit · Review details |
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Member
Author
|
hey @greptileai, please re-review the PR |
This branch was successfully deployed
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.

A design pass over the whole web dashboard: one surface system, one button system, one card system, and a set of layout fixes found by running every page against extreme content.
apps/web/app/globals.css(.dashboard-chrome,.dashboard-sheet,.dashboard-nav-active,.dashboard-nav-hover) with.darkoverrides, so light and dark keep the right contrast withoutdark:utilities.packages/ui: new additivesecondaryButton variant; Popover and Dropdown become white, bordered, elevated menus; Card isrounded-xlwith a subtle shadow; Cmdk input and Popover lose the stray browser focus outline (outline-hiddenis a Tailwind 4 class and does nothing on 3).Shared · 3 days agometa row, a divided stats row, hover scrim and slight thumbnail zoom instead of fading the thumbnail to 30%, square hover buttons, uniform folder card heights, ghost folder menus, segmented layout toggle.Hardening against extreme content
Every page was swept as four synthetic personas (an owner of 14 orgs with 30 spaces, 40 members and 60 Caps with 250-character, unbroken, CJK, Arabic, emoji and URL titles; a brand-new free account; a free account over the link limit; a plain member), at 390, 1024, 1280 and 1440, light and dark, with an automated check for anything overflowing its container.
<aside>is a Tailwindgroup)group/space(SpacesList.tsx)packages/ui/src/components/Select.tsx)smBefore and after
No behaviour changes: every handler, form field, validation, permission check and route is unchanged. Two unit tests had their element lookups updated for the new markup (
browse-spaces-edit,organization-settings-ownership); what they assert is the same.Outside the dashboard, the new Button variant is additive and Card and Popover are dashboard-only. The share page uses the same Dropdown and Select primitives, so its menus pick up the softer shadow and nothing else (left main, right this PR):


Confidence Score: 5/5
The PR appears safe to merge.Summary
The PR gives the dashboard shared surfaces, cards, buttons, and settings rows.
Reviews (2) · Last reviewed commit: "fix(web): keep long folder paths inside ..." · Reviewed by Greptile