ui: fix overview-bar layout, wire About link, add install one-liner#365
Merged
JohnMcLear merged 2 commits intomasterfrom Apr 17, 2026
Merged
ui: fix overview-bar layout, wire About link, add install one-liner#365JohnMcLear merged 2 commits intomasterfrom
JohnMcLear merged 2 commits intomasterfrom
Conversation
…ompiles Two transitive version conflicts were breaking the Deploy etherpad docs to GitHub Pages workflow: 1. \`@fortawesome/free-brands-svg-icons\` was pinned to \`^7.2.0\` while the other \`@fortawesome/*\` deps were \`^7.0.0\`. pnpm resolved two copies of \`@fortawesome/fontawesome-common-types\` (7.0.0 and 7.2.0), and the TypeScript build failed with \`Type 'IconDefinition' is not assignable to type 'IconProp'. ... Type '"fagt"' is not assignable to type 'IconPrefix'\`. 2. \`react\` was \`^19.2.4\` and \`react-dom\` was \`^19.2.0\`, so pnpm resolved them to different minors (19.2.4 and 19.2.0) and Next bailed out with "Incompatible React versions: The react and react-dom packages must have the exact same version". Bump all four \`@fortawesome/*\` packages to \`^7.2.0\` and both \`react\` / \`react-dom\` to \`^19.2.5\` so pnpm resolves a single version of each. \`pnpm run build\` now completes locally. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Three related UI fixes on the home page and header. 1. \`.overview-bar\` items were single flex rows with icon + main text + subtitle all as siblings, so "290 Plugins extend without forking" (and the other three items) rendered with the subtitle inline next to the primary label instead of stacked underneath. Wrap the main text and subtitle in a \`flex-col\` so the subtitle drops below the label the way the rest of the design assumes. 2. The header's \`About\` link scrolled to a no-op \`<a id="about">\` marker on the home page when it should have routed to the dedicated \`/about\` manifesto page. Use Next's \`<Link href="/about">\` directly. Also route the logo back to \`/\` via \`<Link>\` so in-app navigation doesn't trigger a full page reload. 3. Replace the plain "Self-host in 5 minutes" link-to-README button with a new \`<InstallOneLiner>\` component: a tabbed Linux / macOS / Windows switcher that shows the Docker one-liner, a Copy button, and a link to the full install docs. The user can paste the command straight into a terminal without leaving the landing page. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
JohnMcLear
added a commit
to JohnMcLear/ether.github.com
that referenced
this pull request
Apr 17, 2026
Four related tweaks to the home page after PRs ether#361 and ether#365 landed. - InstallOneLiner: swap the Docker one-liner for the native curl/irm installer commands from etherpad-lite README (Linux/macOS use bin/installer.sh, Windows uses bin/installer.ps1). Update the caption from "Needs Docker" to git + Node.js >= 20 with the follow-up `cd etherpad-lite && pnpm run prod` step. - MainHeadline: remove the "Read the manifesto" and "Why Etherpad" CTA buttons from the hero. The header already links to /about, and the hero was getting crowded once the InstallOneLiner landed. - MainHeadline: shrink hero h1 from 2.5rem to 2.3rem so the heading fits more naturally above the subhead on narrower viewports. - AddFunctionalities: rename heading "Add Functionalities" to "Add Functionality" — "functionality" reads as an uncountable collective here, which is more idiomatic. - Footer: remove the unverified Bitcoin donation line that PR ether#361 reintroduced when the positioning branch merged on top of ether#362. No routing or dependency changes. `pnpm run build` succeeds; all routes still generate statically. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
5 tasks
JohnMcLear
added a commit
that referenced
this pull request
Apr 17, 2026
…366) Four related tweaks to the home page after PRs #361 and #365 landed. - InstallOneLiner: swap the Docker one-liner for the native curl/irm installer commands from etherpad-lite README (Linux/macOS use bin/installer.sh, Windows uses bin/installer.ps1). Update the caption from "Needs Docker" to git + Node.js >= 20 with the follow-up `cd etherpad-lite && pnpm run prod` step. - MainHeadline: remove the "Read the manifesto" and "Why Etherpad" CTA buttons from the hero. The header already links to /about, and the hero was getting crowded once the InstallOneLiner landed. - MainHeadline: shrink hero h1 from 2.5rem to 2.3rem so the heading fits more naturally above the subhead on narrower viewports. - AddFunctionalities: rename heading "Add Functionalities" to "Add Functionality" — "functionality" reads as an uncountable collective here, which is more idiomatic. - Footer: remove the unverified Bitcoin donation line that PR #361 reintroduced when the positioning branch merged on top of #362. No routing or dependency changes. `pnpm run build` succeeds; all routes still generate statically. Co-authored-by: Claude Opus 4.7 (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.
Three fixes on the main page and header:
.overview-baritems had their icon, main label, and subtitle rendering as inline siblings instead of a stacked label/subtitle column — the user reported the "290 Plugins extend without forking" formatting looked off. Wrap the label + subtitle inflex-colso subtitles drop below their label the way the rest of the design assumes.<a id="about">marker on the home page when a dedicated/aboutmanifesto page exists. Route the link straight to/aboutvia Next's<Link>. Also restores proper in-app logo navigation to/.<InstallOneLiner/>component — tabbed Linux / macOS / Windows switcher that shows the Docker one-liner with a Copy button and a link to the full install docs.Local
pnpm run buildsucceeds.