[preview] The tornado is five strokes - #314
Open
yayashuxue wants to merge 1 commit into
Open
Conversation
The mark is redrawn: five round-capped lines instead of a traced outline, in assets/tdoc_logo.svg and byte-identical in worker.js. It keeps the two properties the previous mark established — no background field, ink in currentColor — so the bar, the neutral pages and the dark-mode invert all behave as before. What changed is the technique, so the test that guarded paint order and evenodd (artifacts of tracing a raster) now guards `fill="none"` with a currentColor stroke. The viewBox crops to the drawing plus 6%. The source art floats inside a 436 square; carrying that margin would leave the mark about half the box at the 24px bar and the 16px tab slot, which is the lesson the previous mark already recorded. assets/favicon.svg follows the same drawing and keeps its explicit light/dark swap — a tab strip has no page invert to ride on. assets/tdoc_logo_square.svg and tdoc_logo_square_dark.svg are new: the filled square lockup as delivered, for surfaces that bring their own field. Every raster is regenerated from the vector — the Open Graph card and the three home-screen icons from the square, the two off-page rasters from the line art, transparent. Open for review, not to merge on sight: the point is the preview, which now serves the real homepage, so this mark can be judged against tdoc.dev's side by side. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DTsAHQ7Xmdghu9jStyx3WY
PreviewOpen this: https://pr-314-tdoc-preview.jyshi1107.workers.dev/ — the homepage, this PR's code This link is unique to this PR. New commits update the same URL. It is not tdoc.dev. Preview has no Durable Object — concurrent comments use the KV fallback. Data expires in 14 days. |
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.
Opened for the preview, not to merge on sight. Now that a preview serves the real homepage (#313), the redraw can be judged where it ships.
Compare:
The mark is redrawn as five round-capped lines instead of a traced outline, in
assets/tdoc_logo.svgand byte-identical inworker.js. It keeps what the current mark established — no background field, ink incurrentColor— so the bar, the neutral pages and the dark-mode invert behave as before. What changed is the technique, so the test that guarded paint order andevenodd(artifacts of tracing a raster) now guardsfill="none"with acurrentColorstroke.The viewBox crops to the drawing plus 6%: the source art floats inside a 436 square, and carrying that margin would leave the mark about half the box at the 24px bar and the 16px tab slot — the lesson the current mark already recorded.
assets/favicon.svgfollows the same drawing, keeping its explicit light/dark swap.assets/tdoc_logo_square.svg/_square_dark.svgare the filled square lockup as delivered, for surfaces that bring their own field. A filled square reads as a tile against the bar, which is why it is not the in-app mark.Earlier feedback was that the current mark reads more like a tornado — that call stands until this preview says otherwise.
npm testgreen.🤖 Generated with Claude Code
https://claude.ai/code/session_01DTsAHQ7Xmdghu9jStyx3WY