Dw Theme Live Signin Split
repo: ~/Projects/dw-theme-live-signin-split · 6 commits · 0 in last 24h, 1 in last 7d · █
Skills used
- /pages4
- /trade-only-benefits4
- /header2
- /mobile-nav2
- /dw-signin-modal2
- /identical1
- /dw-samples-banner1
- /theme1
- /pxs-announcement-bar1
- /schema1
- /label1
- /trade1
- /register1
Creative ideas + design notes
Commits with substantial prose (≥120 chars) — the rationale behind each move.
26c1c83 · 2026-09-24 · upload_tk11020: require --apply + only ledger a VERIFIED live-theme write
Two guards on a script that PUTs to the live production theme: (1) default to a
no-op dry-run (shows changed/identical) and require --apply, so a bare run can't
silently overwrite the theme; (2) gate the reversible-ledger append on the
post-write verify and sys.exit(1) on failure, so it can no longer record a
"verified" ledger row when the content check actually failed (false-green class).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
f3cd81c · 2026-08-31 · Re-target sign-in split to the PROMO ANNOUNCEMENT BAR (pxs-announcement-bar)
Steve clarified 'top banner' = the promo announcement strip above the header,
NOT the header person-icon chrome.
- sections/pxs-announcement-bar.liquid: bar is now a flex CONTAINER (was one <a>),
holding the promo text (unchanged setting-driven behavior + break gating) PLUS two
entry points: 'Trade Sign In' -> /pages/trade-only-benefits (relative, direct nav,
bypasses modal) and 'Retail Login' -> data-dw-signin (opens retail-only modal).
New schema settings: show_signin_entrypoints, trade_link_text, retail_link_text.
All existing settings/schema preserved (show_announcement, home_only, text, mobile
text, link, colors). Renders on mobile (bar shows on mobile) + desktop.
- sections/header.liquid: REVERTED to live baseline (person-icon back to original
account behavior/label; no Trade Sign In text link in header chrome).
- snippets/mobile-nav.liquid: REVERTED to live baseline (account link back to original).
- snippets/dw-signin-modal.liquid: KEPT (retail-only modal is the Retail Login target).
Verified locally: liquid balanced, schema+header-group JSON valid, both links present,
trade href relative + correct, retail data-dw-signin matches modal's delegated selector,
promo-text settings intact, modal JS node --check passes. NOT pushed (Steve-gated).
00fb8b8 · 2026-08-31 · Split top-banner sign-in: Trade Sign In -> /pages/trade-only-benefits (bypasses modal); Retail Login modal stripped to retail-only with Sign In / Create account / 3 lifetime complimentary samples copy
- header.liquid: add 'Trade Sign In' text link -> /pages/trade-only-benefits next to person-icon; person-icon relabeled 'Retail Login' (keeps data-dw-signin retail modal trigger)
- dw-signin-modal.liquid: remove Professional/trade toggle + trade-apply form; retail-only Existing customer Sign In + Create an account; copy 'Retail accounts receive 3 lifetime complimentary samples'
- mobile-nav.liquid: mobile account link now opens retail modal (data-dw-signin), relabeled 'Retail Login'; existing Trade link untouched
- customer account/register dw_trade redirect: UNTOUCHED (retained for legacy retail->trade edge)
File tree
11 files tracked. Click any to browse the source at HEAD.
rendered in 1ms