Swap Syne for Playfair Display and soften letter-spacing - #7
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1b10cf5bc1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
| .nav { display: flex; align-items: center; justify-content: space-between; gap: 18px; padding: 16px 0; } | ||
| .logo { font-family: Syne, sans-serif; font-size: 1.05rem; letter-spacing: .14em; font-weight: 800; } | ||
| .logo { font-family: "Playfair Display", Georgia, serif; font-size: 1.05rem; letter-spacing: .14em; font-weight: 800; } |
There was a problem hiding this comment.
Load Playfair on every shared-CSS page
Since styles.css is shared by all pages, changing the shared typography to "Playfair Display" only works on index.html; the other routes I checked (about.html, create-profile.html, stories.html, etc.) still load the old Google Fonts URL with family=Syne and never request Playfair. Those pages will render these headings/logo in the Georgia fallback while also downloading an unused Syne font, so the typeface swap is inconsistent outside the home page. Please update the font link on the other HTML pages or load Playfair from the shared stylesheet.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
This PR updates the site’s display typography by switching headings/brand elements from the Syne display font to Playfair Display and reducing overly aggressive negative letter-spacing so large type reads more traditionally.
Changes:
- Swapped the Google Fonts import in
index.htmlfromSynetoPlayfair Display(keepingInter). - Updated
assets/css/styles.cssto apply"Playfair Display", Georgia, serifto key display elements (logo, headings, pricing, quotes, etc.). - Softened negative letter-spacing values for
h1, sectionh2, and.price.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| index.html | Updates Google Fonts link to load Playfair Display instead of Syne (home page only). |
| assets/css/styles.css | Re-points display typography to Playfair Display and adjusts letter-spacing to be less extreme. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <link rel="preconnect" href="https://fonts.googleapis.com"> | ||
| <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> | ||
| <link href="https://fonts.googleapis.com/css2?family=Inter:wght@400;500;600;700;800&family=Syne:wght@600;700;800&display=swap" rel="stylesheet"> | ||
| <link href="https://fonts.googleapis.com/css2?family=Inter:wght@400;500;600;700;800&family=Playfair+Display:wght@600;700;800&display=swap" rel="stylesheet"> | ||
| <link rel="stylesheet" href="assets/css/styles.css" /> |
Motivation
Syne) to a more mature editorial display to increase perceived trust and stability while keeping a clean sans for UI/body text.Description
SynewithPlayfair Displaywhile retainingInterfor body/UI inindex.html.assets/css/styles.cssto use"Playfair Display", Georgia, seriffor.logo,h1,h2,.metric b,.price,.quote, and.owner-mark span.h1, sectionh2and.priceto gentler values so large display text feels more established.font-family: Interunchanged so UI and paragraph text remain a clean sans.Testing
git diff --checkwith no reported issues and it passed.python3 -m http.server 8000and verified the updated font link viacurl -I http://127.0.0.1:8000/index.htmlwhich returned HTTP 200.curl -s http://127.0.0.1:8000/index.html | rg "Playfair|Syne", which showedPlayfairand no remainingSynereferences.Codex Task