Open
Conversation
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Establishes project conventions for the Golden Horizon theme and documents the approved 3-phase refactor plan: strip unused files, tokenise with Worker design system, then SEO + performance fixes. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Three-phase plan: strip 29 dead files, tokenise with Worker --space-* scale, then SEO schemas + JS deferral. Verified all deletions are safe via grep audit of render/include references. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove 17 unused blocks, 11 unused snippets. All deletions verified safe via grep audit — no render/include calls reference these files. Blocks removed: _collection-image, _collection-info, _content, _footer-social-icons, _inline-text, _media, _social-link, accordion_debug, accordion_image, blog_card, custom-liquid, featured-collection, page, popup-link, product-custom-property, product-inventory, vertical-text Snippets removed: filters-toggle, 9x special-accordion-preset-*, split-showcase-carousel Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
tokens-base.css: shared foundation (spacing, type, motion, layout) tokens-theme.css: Golden brand overrides (colours, fonts, radius) Loaded before base.css so existing styles take precedence for now. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
All --margin-*, --padding-*, --gap-* variables now reference the unified --space-* scale from tokens-base.css. Old variable names preserved as aliases for backward compatibility. Some values snap to the nearest scale step (e.g. 0.7rem -> 0.75rem, 0.8rem -> 1rem). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
86 replacements: all var(--margin-*), var(--padding-*), var(--gap-*) references in base.css now use the unified --space-* scale. Section layout vars (--padding-block-start etc.) left untouched as they serve a different purpose (section settings pipeline). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- #e1b87f -> var(--color-product-honey) - #fff backgrounds -> var(--color-surface) - #000 box-shadow -> var(--color-ink) 3 remaining hex values are CSS var() fallbacks — intentionally kept. All font-family declarations already use CSS variables. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Organization schema with social links (Instagram, TikTok, LinkedIn, Threads) - FAQ schema snippet for accordion-based FAQ pages - Sitemap link tag in meta-tags - theme-color set to Golden pink (#f8b5c4) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Homepage: visually-hidden h1 with shop name in logo block - Collection pages: collection title now uses h1 instead of p - Product pages: already have h1 via product title (no change) - Content pages: already have h1 via main-page section (no change) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Product scripts (variant-picker, product-card, product-rotate, product-form, product-price, product-inventory, media-gallery, gift-card-recipient-form) now only load on pages that need them. Quantity selector only loads on product and cart pages. Content pages (FAQ, Our Story, etc.) no longer download these scripts at all, reducing their JS payload. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Committed from shop: Golden
Typography tokens in theme.css (measure, leading, tracking for display/ heading/caps). Butterick baseline in base.css: font kerning, optimised text rendering, prose max-width on .rte content, tabular numerals for prices, caps letterspacing rule, display text tracking, iOS input zoom prevention. Fixed heading margin values in theme-styles-variables.liquid. Gangster Grotesk confirmed to support macron characters (te reo Maori). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
Committed from shop: Golden
9 tasks across 3 chunks: baseline + cleanup, CSS rewrite, verification + rename. Fresh-write CSS strategy approved in eng review. Conservative deletion list (verified references). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Performance: 42, Accessibility: 88, Best Practices: 57, SEO: 100. LCP 4.8s, TBT 2,660ms, CLS 0. This is the "before" snapshot for measuring the impact of the deep fork. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
13 blocks, 11 sections, 5 JS files confirmed to have zero references in any Liquid file, template JSON, or layout file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Performance: 42 → 75 (+33). TBT: 2,660ms → 90ms. LCP: 4.8s → 4.1s. Added performance budget targets to CLAUDE.md. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Updated CLAUDE.md to reflect the new name and forked identity. GitHub repo renamed from drinkgolden/horizon to drinkgolden/hour. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
JS consolidation (P2), LCP optimisation (P3), Best Practices audit (P3). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Took remote versions of header-group.json and footer-group.json as these are managed by the Shopify admin editor. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.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
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.
No description provided.