Skip to content

[2/4] Replace trace viewer inline styles with Tailwind - #3308

Draft
mitul-s wants to merge 3 commits into
c+ms/remove-inline-styles-86b1from
c+ms/trace-viewer-tailwind-86b1
Draft

[2/4] Replace trace viewer inline styles with Tailwind#3308
mitul-s wants to merge 3 commits into
c+ms/remove-inline-styles-86b1from
c+ms/trace-viewer-tailwind-86b1

Conversation

@mitul-s

@mitul-s mitul-s commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Replace static inline styles in the trace viewer with Tailwind utilities.
  • Map resource colors to Geist theme class names.
  • Keep runtime timeline geometry inline.

Stack

  1. Shared UI components
  2. This PR — trace viewer
  3. Workflow web/graph
  4. Workbench code block

Merge in order from 1 to 4.

How did you test your changes?

  • pnpm exec turbo build --filter=@workflow/web-shared...
  • pnpm --filter @workflow/web-shared exec vitest run --exclude test/zstd-decoder.test.ts — 157 tests passed
  • Biome check on all 5 changed source/test files — no errors

The excluded zstd tests require node:zlib.zstdCompressSync, which is unavailable in the local Node 22 runtime.

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
  • 🔒 DCO sign-off passes (git commit --signoff)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
Open in Web Open in Cursor 

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
example-nextjs-workflow-turbopack Ready Ready Preview Aug 3, 2026 9:01pm
example-nextjs-workflow-webpack Ready Ready Preview Aug 3, 2026 9:01pm
example-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-astro-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-express-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-fastify-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-hono-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-nestjs-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-nitro-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-nuxt-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-sveltekit-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-tanstack-start-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workbench-vite-workflow Ready Ready Preview Aug 3, 2026 9:01pm
workflow-docs Ready Ready Preview, v0 Aug 3, 2026 9:01pm
workflow-swc-playground Ready Ready Preview Aug 3, 2026 9:01pm
workflow-tarballs Ready Ready Preview Aug 3, 2026 9:01pm
workflow-web Ready Ready Preview Aug 3, 2026 9:01pm

@changeset-bot

changeset-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2d6266a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@workflow/web-shared Patch
@workflow/web Patch
@workflow/cli Patch
@workflow/nitro Patch
workflow Patch
@workflow/world-testing Patch
@workflow/nuxt Patch
@workflow/core Patch
@workflow/builders Patch
@workflow/next Patch
@workflow/vitest Patch
@workflow/astro Patch
@workflow/nest Patch
@workflow/rollup Patch
@workflow/sveltekit Patch
@workflow/vite Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
…into c+ms/trace-viewer-tailwind-86b1

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursor cursor Bot changed the title Replace trace viewer inline styles with Tailwind [2/4] Replace trace viewer inline styles with Tailwind Aug 3, 2026
@cursor
cursor Bot changed the base branch from main to c+ms/remove-inline-styles-86b1 August 3, 2026 20:59
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.

2 participants