Skip to content

fix(site): center the landing grids, hide shouting-titles on the hack… - #34

Merged
juliojimenez merged 1 commit into
mainfrom
julio/site-updates-2026-11-04
Nov 5, 2026
Merged

juliojimenez merged 1 commit into
mainfrom
julio/site-updates-2026-11-04

Conversation

@juliojimenez

Copy link
Copy Markdown
Member

…ernews demo, trim the footer

What / why

Four fixes to the landing site; no compiler, rules or UDF changes.

  • Centered the two 2x2 card grids ("Built for queries that are programs" and "Rule packs running on real data"). Both blocks sat visibly left of their centered headings. Two causes: Vite code-splits click-ui's per-component CSS into the landing page's own stylesheet, which loads after styles.css, so .feature-grid { max-width: 940px } lost the equal-specificity tie against GridContainer's own max-width and the grid spanned the full 1200px; and CardSecondary clamps itself to 320–420px, so each card start-aligned inside its oversized cell. The cap now goes through GridContainer's maxWidth prop (an inline custom property, immune to stylesheet order), and .feature-grid .feature-card lifts the card clamp so each card fills its cell. The 620px media query is gone because GridContainer already collapses to one column at ≤768px. The CSS comment records the stylesheet-order trap.
  • Hackernews demo: dropped the shouting-titles card. It cannot run on the public playground (it needs the local clicklisp_shout executable UDF) and only ever rendered a "not runnable here" badge. Demo.nonRunnable is replaced by Demo.hiddenRules, which DemoLayout filters on, and the now-dead badge branch is removed from RuleCard. The query stays in examples/analytics/hackernews.lisp and in the generated JSON; README's local-ClickHouse section still covers it and gains one clause saying UDF-dependent queries stay off the demo pages.
  • Footer: removed the click-ui link from the ClickHouse column. The bottom bar already credits @clickhouse/click-ui; that surviving link now opens in a new tab, as the removed one did and as every other footer link does.

Verified in headless Chrome against the production build and under vite dev: at 1400px both grids are 940px wide with 130px on each side and four equal 462px cards; one full-width column at 760px and 375px with no overflow; the hackernews page lists hype-tracker, long-words and prolific-authors; the ClickHouse footer column has three links.

Not touched, pre-existing and unrelated: below ~700px the page scrolls horizontally because the Get started code pane inside the click-ui Panel does not shrink. Worth a separate fix (width: 100%; min-width: 0 on that pane).

Checklist

  • make test and make smoke pass
  • README.md and site/ updated if behavior or docs changed (samples in site/src/samples.ts stay verbatim)
  • Tests added/updated for compiler, rules, or UDF changes (n/a: site-only change)
  • Code stays portable to ECL 21.2.1 (no Lisp changes)

🤖 Generated with Claude Code

…ernews demo, trim the footer

Signed-off-by: Julio Jimenez <julio@julioj.com>
@juliojimenez juliojimenez self-assigned this Nov 5, 2026
@juliojimenez juliojimenez added the fix Fixes a bug label Nov 5, 2026
@juliojimenez
juliojimenez merged commit db6b014 into main Nov 5, 2026
6 checks passed
@juliojimenez
juliojimenez deleted the julio/site-updates-2026-11-04 branch November 5, 2026 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant