Skip to content

docs: add the taskito logo - #582

Merged
kartikeya-27 merged 5 commits into
masterfrom
docs/logo
Jul 29, 2026
Merged

docs: add the taskito logo#582
kartikeya-27 merged 5 commits into
masterfrom
docs/logo

Conversation

@kartikeya-27

@kartikeya-27 kartikeya-27 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Puts the taskito logo on the docs site, the README, and the dashboard, and gives all three a real favicon (none of them had one).

What landed where

Docs sitelogo.png replaces the CSS-drawn .logo square in the nav and footer. .logo now sizes on height (height: 27px; width: auto): the artwork is 770×513, so the old 27×27 square box would have cropped the flanking queue
arrows. Favicon, apple-touch-icon and web manifest wired in root.tsx, all through import.meta.env.BASE_URL so they resolve under the /taskito Pages base rather than at the domain root.

README — the logo above the title, and the title itself replaced with a two-tone wordmark matching the site's task + accented ito. GitHub strips inline styles from markdown, so the accent has to ship as an image; light and dark variants swap via prefers-color-scheme. Colors are the docs tokens (#13a44e light, #3bbf72 dark).

DashboardBrandMark was a hand-drawn rounded square with dot eyes and a check-smile, unrelated to the actual logo. Now the same artwork, plus favicon links in index.html.

Three judgement calls

favicon.svg is not included. The generated bundle ships one, but it holds a single <image> element wrapping a base64 raster — zero <path> elements, so it is a PNG in SVG clothing at 294 KB. Browsers prefer type="image/svg+xml" over .ico when both are declared, so including it would cost every tab load 294 KB instead of the 15 KB .ico.

site.webmanifest was rewritten. It shipped as "name": "MyWebSite", "short_name": "MySite", white theme colors, and absolute /web-app-manifest-*.png icon paths — the last of which 404s under the /taskito base. Now taskito, relative icon paths (resolved against the manifest URL, correct at any base), and #364b53 sampled from the mark's hexagon stroke.

The dashboard mark no longer follows brandAccent. The drawn mark used var(--accent) / var(--accent-fg), so it tracked an operator's Settings → Branding accent override; fixed artwork cannot. The sidebar wordmark beside it still tracks the accent and the custom-title path is untouched, but an operator who rebrands the accent now sees taskito-green artwork next to their color. Noted in the component docstring — say the word and I'll keep the drawn mark whenever brandAccent is set.

Verification

  • docs: biome + typecheck clean; DOCS_BASE_PATH=/taskito build emits /taskito/favicon.ico, /taskito/favicon-96x96.png, /taskito/apple-touch-icon.png, /taskito/site.webmanifest, /taskito/logo.png
  • dashboard: biome + typecheck clean, 12/12 files and 137/137 tests
  • wordmark rasterized headless with the fallback mono font (IBM Plex Mono will not be installed for most README readers) to confirm it fits its viewBox
  • dashboard CSP is img-src 'self' data:, so the bundled PNG passes
  • the built SPA under sdks/*/static/dashboard is untracked, so no committed artifact goes stale; build:dashboard regenerates it

Summary by CodeRabbit

  • New Features
    • Updated branding across the dashboard and documentation site with the new logo and responsive wordmark.
    • Added favicon, Apple touch icon, and web app manifest support.
    • Improved asset paths for deployments under subdirectories.
  • Documentation
    • Refreshed the README header with centered branding and badges.
  • Style
    • Simplified logo sizing and presentation for consistent display across layouts.

Excludes the generated favicon.svg — it wraps a base64 raster, so browsers would prefer 294K over the 15K .ico.
GitHub strips inline styles, so the accent on "ito" has to ship as an image; light/dark variants swap via prefers-color-scheme.
Trade-off: the drawn mark tracked the operator's brandAccent override; the artwork has fixed colors.
@kartikeya-27 kartikeya-27 added the documentation Improvements or additions to documentation label Jul 29, 2026
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The change standardizes branding across the dashboard, documentation site, and README, replacing placeholder or inline logo rendering with raster assets and adding base-aware favicon, manifest, and icon references.

Changes

Branding and site assets

Layer / File(s) Summary
Documentation site asset wiring
docs/app/root.tsx, docs/public/site.webmanifest
Documentation icon and manifest URLs now include the deployment base path, and a standalone web app manifest defines site metadata and maskable icons.
Documentation site logo rendering
docs/app/components/landing/footer.tsx, docs/app/components/ui/site-nav.tsx, docs/app/styles/atoms.css
Footer and navigation logos now use logo.png, with shared CSS updated for height-based sizing.
Dashboard and README branding
dashboard/src/components/layout/brand-mark.tsx, dashboard/index.html, README.md
The dashboard uses the raster logo and additional favicon links, while the README uses a centered logo and badge header.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is concise and accurately reflects the main change: adding the taskito logo across docs-related surfaces.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/logo

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Consecutive raw HTML blocks inside the centered div render inline, so the two marks sat side by side; explicit paragraphs separate them.
@kartikeya-27 kartikeya-27 changed the title docs: add the taskito logo across docs, README, and dashboard docs: add the taskito logo Jul 29, 2026
@kartikeya-27
kartikeya-27 merged commit 5508015 into master Jul 29, 2026
23 checks passed
@kartikeya-27
kartikeya-27 deleted the docs/logo branch July 29, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard docs documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants