v1.67.0 - #812
Merged
Merged
Conversation
…ditor contract (#811) * feat: replace template lineup with 10 ICP-driven themes Rebuilds all templates from a study of the paying customer base (decision trees + process flows = 78% of payers; swimlane and timeline demand hand-faked across every persona): - default (new sandbox first-load look), decision-tree, process-lanes, blueprint, ink-mindmap, constellation, org-chart, sitemap, timeline, storyline - modern cytoscape techniques as pure template data: corner-radius cards, round-taxi elbows, underlay shadows/glows, edge label pills, data-URI SVG icon classes, gradient star nodes - 7 self-hosted open-license font families (app/public/fonts) wired into fonts.ts and the theme editor font picker - toTheme: round-taxi now receives taxi-direction like taxi - screenshot script honors E2E_START_URL - visual regression goldens + frozen fixtures regenerated for the new lineup; toTheme characterization baselines updated deliberately Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix: keep every theme-editor knob effective in all templates Cytoscape resolves style ties by rule order, not specificity, so a template cytoscapeStyle rule that sets an FFTheme-controlled property on a broad selector (node/:childless/edge/:parent) silently pins it and breaks the matching Theme-tab knob — e.g. changing the font did nothing because every template shipped a 'node { font-family: ... }' rule. - remove the node font metric rules (render-neutral: measured sizes are only consumed by the [w]/[h] selectors) - stop overriding parent background-color and edge text-background-color in bare rules (the background knob owns these); lane/zone tints remain via :parent.color_* classes applied in starter content - constellation: express star size, border, text color and margin through the theme object (useFixedHeight 26, nodeForeground, etc.) instead of pinning them in css; drop curve-style override Adds themeEditorContract.test.ts: for all templates x 22 FFTheme knobs, composes the full pipeline stylesheet exactly like the app and asserts against a headless cytoscape instance that changing the knob changes the computed style of a plain node/edge/parent. Class- and special-selector styling (color_*, icon_*, degree/label selectors) stays exempt by construction. Visual goldens, frozen fixtures and template screenshots regenerated; verified in-browser that picking a font in the Theme tab restyles the graph. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * feat: curate font picker with SVG previews; add Skriva, retire legacy fonts - fonts.ts split: 'fonts' (picker: the 7 repertoire sans + Skriva) vs 'legacyFonts' (IBM Plex Sans, Inclusive Sans, Space Grotesk, Onest, Overpass, Kalam, Nanum Pen Script, REM). Legacy names leave the picker but stay loadable via allFonts — existing customer charts have them baked into meta.themeEditor and must keep rendering unchanged. - Skriva (single handwritten voice) replaces Kalam + Nanum Pen Script: self-hosted otf + css. NOTE: Skriva is a licensed family — confirm the webfont license before this deploys publicly. - font picker previews: scripts/generate-font-previews.mjs (pnpm -F app font:previews) pre-renders each picker font's name to an SVG path via opentype.js; the dropdown shows the preview image (dark-mode via invert) so no font file loads until a font is chosen. - fonts.test.ts guards the contract: every picker font has a preview + css asset; every historical font name still resolves its @import through toTheme; no picker/legacy overlap. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * fix: font picker shows selection in its own face; stop form-submit reload Three fixes found while polishing the picker: 1. The closed control now overlays the selected font's SVG preview (input text goes transparent while not editing), so the selection is shown in its own face instead of the UI font. Typing custom font names still works — the overlay yields to plain text on focus. 2. Pre-existing bug: FontpickerButton had no type attribute, so picking a font SUBMITTED the surrounding theme-editor <form> as a GET navigation — a full page reload on every font choice (the change only survived via localStorage). Now type="button", plus a preventDefault guard on the form itself. 3. generate-font-previews serializes glyph paths by hand: opentype.js' Path.toPathData() emitted a literal NaN at certain offsets (Skriva's 'i' dot), and one bad token makes SVG parsers drop the rest of the path — Skriva's preview rendered as 'Skri'. Also composes previews glyph-by-glyph (string shaping routed through GSUB alternates that opentype.js can't extract) and uses static Satoshi Regular for the preview so it isn't rendered from the variable file's heavy default. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * feat: woff2 fonts + frozen legacy default for pre-metadata charts Two mitigations for shipping the theme overhaul: 1. Shipped fonts converted to woff2 (1.9MB of otf/ttf -> 968KB; e.g. Satoshi 128KB ttf -> 44KB woff2). The otf/ttf sources stay in public/fonts as inputs for generate-font-previews (opentype.js can't read woff2) but nothing references them at runtime. fonts.test.ts now also verifies every binary referenced by a font css file exists. 2. ~1,700 hosted charts (513 owned by active subscribers) predate baked theme metadata; prepareChart backfills a default theme for them on open. That fallback is now a FROZEN copy of the pre-overhaul default (legacyDefaultTheme.ts) instead of the current default template, so existing documents keep rendering exactly as their owners last saw them — including on public /p/ links. New documents are unaffected (getDefaultChart bakes the current theme at creation). A freeze-guard test fails if the fallback is ever re-pointed at the evolving default. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * feat: replace Skriva with Shantell Sans (OFL) as the handwritten voice Skriva is a commercial family we can't license right now; Shantell Sans is SIL Open Font License with genuine handwriting character — zero licensing blockers for deploy. Self-hosted from the google/fonts repo, instanced down to the weight axis only (the BNCE/INFM/SPAC novelty axes tripled the file): 2.1MB source -> 208KB woff2. The weight-instanced ttf stays in public/fonts as the preview-generator source. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
|
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.
v1.67.0
New: theme overhaul — all 13 templates replaced with 10 job-shaped themes, plus a new first-load look for every visitor (#811).
🤖 Generated with Claude Code