Typography - #78
Merged
Merged
Typography#78
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
WebhooksPage and its owned same-dir sub-components (WebhookSummary, WebhookGuide) already render every text node through the shared typography primitives/tokens — PageContainer title/subtitle, PanelTitle, Text variants (caption/bodySm/label), Code, Badge, and MetricCard — with theme --text-* colors and no ad-hoc size/weight/font-family or non-theme color overrides. No §2 auto-fail patterns and no role+className overrides to normalize (only layout utilities like break-all/shrink-0 remain). Typography audit clean (0 violations), eslint clean, page gate PASS (line-count ratio 1.0). No source change required; empty marker commit records the sweep, matching the Quiet Hours sweep precedent. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The 141-display-modes merge reintroduced two text-[10px] arbitrary sizes in the ThemePicker mode-count + category labels; align them to the Tailwind scale (text-xs) so the typography audit stays green. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
System Status and Settings laid out varying-height panels in a CSS grid with items-start, so each row was as tall as its tallest card and short cards left large dead space beneath them. Introduce a shared Masonry (CSS multi-column) layout in components/layout and use it for the card containers so cards pack tightly with even, consistent spacing. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Same uneven-gap fix as System Status/Settings: the variable-count category cards were laid out in a grid with items-start, leaving dead space under shorter categories. Use the shared Masonry component so they pack tightly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The per-tour 'seen' flags lived only in localStorage, so clearing cookies/
site-data re-triggered the intro tour every time. Persist them in the settings
KV store (completed_tours JSONB array of '{tourId}:{version}' tokens) so
completion survives a clear and syncs across devices — same round-trip as
theme/units through GET/PUT /api/v1/settings.
Backend: completed_tours on systemmodel.Settings, JSONB KV row + migration
000214, handler normalization (trim/dedupe/cap 100), unit tests.
Frontend: seed local flags from settings on load, gate tour auto-start until
settings are fetched, persist to the DB on finish/skip (+ optimistic cache),
mirror peer-tab completions.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The Appearance panel was one giant card (~half the page) holding theme, density, sidebar, time format, chart palette, status bar, celebration and tours — so it dominated one masonry column and left the other short, producing an awkward gap. Break it into four cards (Theme / Density+Sidebar+Time / Charts+Status / Celebration+Tours) so the masonry balances both columns. The '#appearance' deep-link anchor moves onto the first card (GlassPanel forwards id). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The ~260MB Windows Docker build context (apps/, scripts/, bin/, backups/,
docs/node_modules + .vitepress, stray node_modules) was canceling mid-transfer
('error from sender: context canceled'). None are needed by any build stage;
docs/public/openapi.yaml is retained. Shrinks context to ~30MB and makes the
teslasync-api image build reliable.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The Button 'primary' variant hardcoded bg-blue-600, so changing the accent color (Neon Cyan, Tesla Red, …) did nothing to buttons app-wide. Use var(--theme-primary) for the fill and a luminance-computed var(--theme-on-primary) for the foreground (dark text on bright accents, white on dark) so buttons stay readable across every accent. ThemeProvider now sets these vars in applyThemeCSS. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
Description
Closes #
Type of Change
Checklist
Screenshots (if applicable)