Worktree formatters dashboard alignment - #236
Merged
Merged
Conversation
Scopes the 9-tool "Formatters" category to inherit dashboard's visual system (icon-chip headers, gradient cards, motion, ambient bg) via shared components, rather than editing each tool file independently. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
13 tasks: 3 shared components (ToolPageHeader, ToolMobileTabs, CopyIconButton), then pilot on JSON formatter, then batch-apply to the remaining 8 tools in the Formatters category, then a cross-tool QA pass. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Implements a reusable mobile input/output switcher for tool pages that wraps shadcn's Tabs primitive, replacing hand-rolled pill-button toggles with a consistent component used elsewhere in the app (e.g. dashboard, markdown preview). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Pilot for the tool-page-shell refactor: replaces the JSON formatter's inline mobile-only header card and hand-rolled tab-switcher buttons with the shared ToolPageHeader and ToolMobileTabs components, wraps the page in the dashboard ambient-background shell, and adds RevealItem mount animation. Resizable dual-pane editor and save/load modal are untouched.
apps/web imports it directly in three pages (database-explorer, redis-commander, sql-client) but only had it as an undeclared transitive dependency. A fresh pnpm install (as in this worktree) doesn't hoist it, breaking the entire /app/* route tree at runtime since those pages are bundled transitively via the shared tab registry. The main checkout's stale node_modules happened to mask this. Isolated to this worktree/branch; needed to run the dev server for manual verification of the formatters-alignment tasks.
Replace the hand-rolled mobile header, tab switcher, and copy button in the YAML formatter with the shared ToolPageHeader, ToolMobileTabs, and CopyIconButton components, matching the pattern established on the JSON formatter pilot.
Replaces the SQL formatter's hand-rolled mobile-only header, pill-button mobile tab switcher, and inline copy button with the shared ToolPageHeader, ToolMobileTabs, and CopyIconButton components already introduced for the JSON and YAML formatters, keeping the tool pages visually consistent.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
… siblings Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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.
No description provided.