Repository navigation
Apply the homepage design review - #92
Merged
Merged
Conversation
Implements the 14 fixes from the design review of luinbytes.dev: pond controls demoted to a floating chip, single click pond feeding with no right-click hijack, page-wide text selection restored, the leftover purple glass palette removed, shadcn Button removed from project nav, type floor raised, Instrument Serif self-hosted for display type, all four projects visible in a grid instead of the tabbed explorer, visible email CTA and calendar icon for Cal.com, repeated copy rewritten, dark mode contrast fixed, one shared container width, wider section spacing, and head cleanup (title, theme-color, SVG favicon, og alt text).
Owner
Author
|
bugbot review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d330cf9. Configure here.
luinbytes
commented
Nov 10, 2026
luinbytes
left a comment
Owner
Author
There was a problem hiding this comment.
approved?????????????????????/
5 of 7 tasks
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.

Implements the fixes from an Opus 5.5 design review of the homepage (luinbytes.dev only). The pond and the site's identity are unchanged in spirit; this is about hierarchy, legibility and removing the leftover second design system.
What changed
user-select: noneis gone; selection is only disabled on the pond surface itself.--site-bg: #252031and friends) is deleted. Page background, pond edge, skip link, scrollbar and text selection now use the green and paper palette the page actually shows, in light and dark.--secondaryreferences from this section.#3d5e45(about 1.95:1 on the dark background) to a passing value, dark hairlines lightened, and the hard-coded hero shadow and card colours are tokenised.min(100% - 96px, 1200px); the footer gets its missing top border.Notes and trade-offs
app/fontsexactly as before. Instrument Serif is embedded inglobals.cssas a data URI rather than apublic/fontsfile, and font preloading was dropped in favour of this, because this branch was pushed through an API that cannot carry new binary files. If you would rather have the woff2 as a real file with preloads, it is a two minute follow-up commit.npm run buildpasses (type check, client build, SSR prerender of all 3 pages). The e2e suite was not run.Note
Low Risk
Changes are mostly presentational CSS, static content, and client-side pond/input behavior on a marketing site; no auth, data, or backend logic.
Overview
This PR implements a homepage-focused design pass: hierarchy, one visual system (green/paper), typography, and UX polish, without changing the pond’s core idea.
Visual system & fonts: Global purple “glass” tokens are removed from
globals.cssand kept only on the 404 page. The homepage repoints shared--site-*tokens viasignal-desk.module.cssso skip link, scrollbar, and selection match the pond palette. Instrument Serif is added (OFL license file + ~21 KB Latin subset as a data URI in CSS) for display headings via--font-serif-display, with larger type scale andem-based tracking.Layout & projects: Pond view/feed controls move from the hero to a fixed bottom-corner chip (safe-area aware), shrinking mobile hero padding. The tabbed project explorer becomes a 2×2 card grid with headline figures; shadcn
Buttonis dropped for native-styled links.Pond interaction: Feeding is one click/tap (splash + food); double-tap timing and context-menu handling are removed; selection is disabled only on the pond surface.
Content & metadata: Copy/CTAs refresh (visible email + copy button, nav “More”, deduped messaging). Route metadata gets the new title, per-scheme
theme-color, and richer OG alt text; HTML shells addviewport-fit=cover, SVG favicon, and apple-touch-icon.Responsive polish: Shared
--container/ section gaps, dark-mode contrast tweaks, and safer-area insets on header, skip link, and footer spacing for the floating chip.Reviewed by Cursor Bugbot for commit de72687. Bugbot is set up for automated code reviews on this repo. Configure here.