Major revamp of the Playground#138
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
tstirrat15
force-pushed
the
new-play
branch
3 times, most recently
from
c8409ea to
a2f09c3
Compare
…ore baselines, core hooks - Install shadcn/ui primitives (Dialog, Tooltip, Dropdown, Tabs, etc.), Zustand, jsdom, Inter font - Add cloud-dashboard Tailwind tokens; drop CRA boilerplate; switch to Inter - Unify Monaco editor: shared theme, multi-editor instance support, manual layout observer - Track datastore baseline for "modified" detection - Add useDocumentIdentity + useResolvedTheme hooks - Add top-bar chrome components (auth-slot, breadcrumb-pill, document-link, theme-toggle)
…ed wrapper to shadcn - Migrate panels and old coordinator to shadcn primitives - Migrate relationship editor cells + grid to shadcn - Migrate playground-ui internals to shadcn; delete PlaygroundUIThemed and theme.ts - Migrate misc components to shadcn / Tailwind - Scope the Material UI ThemeProvider to embedded mode (/e/) only
tstirrat15
force-pushed
the
new-play
branch
2 times, most recently
from
89c6aa9 to
4fbf65e
Compare
tstirrat15
reviewed
May 11, 2026
Comment on lines
-26
to
-27
| - name: "Run Typescript type checking" | ||
| run: "yarn run typecheck" |
Contributor
There was a problem hiding this comment.
The lint step already does typechecking
| test: | ||
| name: "Run Tests" | ||
| runs-on: "depot-ubuntu-24.04-small" | ||
| runs-on: "depot-ubuntu-24.04-4" |
Contributor
There was a problem hiding this comment.
More beef since we're running chrome
Comment on lines
-18
to
-20
| - uses: "authzed/action-spicedb@v1" | ||
| with: | ||
| version: "latest" |
Contributor
There was a problem hiding this comment.
Not needed when everything is wasm
Contributor
There was a problem hiding this comment.
Moved things to vite browser tests
| <TooltipTrigger asChild> | ||
| <PanelButton | ||
| id="problems" | ||
| aria-label="problems panel trigger" |
Contributor
There was a problem hiding this comment.
Added a label for testing purposes
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
Summary
Rebuilds the playground UI on shadcn / Tailwind, replaces the Material UI / Reflex chrome with a cloud-style layout (editor groups, bottom drawer, status strip), adds shareable +
persistable check watches, and ships a settings dialog.
What changes
Foundation
useDocumentIdentity,useResolvedThemeLayout & redesign
FullPlaygroundrestructured around the new chromeInlinePlaygroundredesigned as a docs-first viewer/e/) onlyCheck watches: share / persist / load
check_watchescheckWatchesfieldFullPlaygroundandEmbeddedPlaygroundEditor-groups follow-ups
Schema
showDocumenteditor actionSettings dialog
EditorDisplayreads minimap from settings only whenhideMinimapis undefined; embedded override preservedPolish & fixes
ValidateButtonheight aligned with the rest of the toolbarDialogOverlaywrapped inforwardRefsoDialogPortal's Slot.cloneElement doesn't warn