Clay + 3D visual redesign (Slate & Coral palette, dark mode) - #4
Open
SM649 wants to merge 24 commits into
Open
Conversation
- Add CSS custom properties (--clay-*) for light and dark modes in static/tailwind-input.css - Enable class-based dark mode in tailwind.config.js - Rewrite Tailwind theme to use CSS variables for all colors, shadows, and spacing - Remove unused tokens (primary-darker, gradient-start, gradient-end, brand-gradient-h variants, text-medium) - Add .clay-blob component class for decorative hero shapes - Rebuild static/tailwind.css with new configuration Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- Add anti-flash localStorage script to both base templates - Implement toggleClayTheme() function to toggle dark class and persist to localStorage - Add toggle buttons to dashboard topbar and public navbar (desktop + mobile) - Rebuild tailwind.css to include any new tokens - Replace profile dropdown's bg-white with bg-surface for theme awareness Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Rebuilt static/tailwind.css using the correct command with --minify: ./.tailwindcss-cli -i static/tailwind-input.css -o static/tailwind.css --config tailwind.config.js --minify This reduces file size from 27KB to 18KB and removes formatting noise. All Tailwind classes and custom tokens verified present in minified output. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- Swap card background from bg-white to bg-surface in both templates - Add rounded-t-brand-lg to card header bands for consistent corner treatment - Rebuild tailwind.css to include new rounded-t-brand-lg utility Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…ad input-focus token
…ents, landing preview, history rework)
…t history, fix truncate, reduced-motion fallback, search empty state, honest copy Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Owner
Author
Structure & UX follow-up (10 new commits)Added actual structural/UX improvements on top of the visual redesign above:
Follow-ups for a maintainer (not code fixes — flagged for visibility)
🤖 Generated with Claude Code |
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.
Summary
.clay-blobshapes on hero sections.loading="lazy"to chart/thumbnail images.bg-positive/10,bg-primary/20, etc.) because their colors were barevar()hex values — converted to RGB-triplet +<alpha-value>form. Also fixed a footer/dark-mode background collision and removed a dead config token.Known follow-ups (not fixed in this PR — flagged for a maintainer decision)
#ff7a72) on white text is ~2.54:1 contrast, below WCAG AA (4.5:1) — a regression from the old palette's 6.13:1. Affects primary CTA buttons.create_bar_chart.py) render on an opaque white background and show as white boxes in dark mode — fixing this needs a Python change, out of scope for this template/CSS-only PR.loading="lazy"on base64 data-URI images doesn't achieve real deferral (no network request to defer) — harmless but doesn't meet the originally-stated perf rationale.rel=preconnectfor Google Fonts was not added (spec item, missed).task9testuser...) was created in the live Firestoreusers_datacollection during manual verification and was not cleaned up (no delete-account route exists) — should be removed manually.Test plan
static/tailwind.cssrebuilt and verified minified after every token/class-affecting change/,/about,/login,/register, dashboard/,/profile_settings,/history(empty-history branch)/historypage, Network-tab lazy-load check (not exercisable in this environment)🤖 Generated with Claude Code