Skip to content

Update website design - #83

Merged
offyotto merged 1 commit into
mainfrom
feat/fable-5-website
Sep 9, 2026
Merged

Update website design#83
offyotto merged 1 commit into
mainfrom
feat/fable-5-website

Conversation

@offyotto

@offyotto offyotto commented Sep 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

No description provided.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@offyotto offyotto changed the title Refresh website with Fable 5 design new website Sep 9, 2026
gitar-bot[bot]
gitar-bot Bot reviewed Sep 9, 2026
View reviewed changes
Comment thread docs/styles.css
@gitar-bot

gitar-bot Bot commented Sep 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Code Review ✅ Approved 1 resolved / 1 findings

Updates website design, addressing the fan curve visibility issue without JavaScript. No open issues.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

✅ 1 resolved
✅ Bug: Fan curve stays invisible without JS for most visitors

📄 docs/styles.css:397-411 📄 docs/script.js:4-6 📄 docs/script.js:83-97
The initial hidden state of the fan-curve SVG (.curve-line { stroke-dashoffset: 620 }, .curve-node/.curve-op { opacity: 0 }) lives inside @media (prefers-reduced-motion: no-preference), and it is only revealed by the .is-drawn class that script.js adds. So for the majority of visitors (no reduced-motion preference), if script.js is blocked, fails, or hasn't run, the curve line, nodes and operating point never appear. This directly contradicts the progressive-enhancement promise in the script.js header comment ("With this file blocked the page stays fully readable: the fan curve renders fully drawn"). Fix by making "drawn" the default and gating only the un-drawn starting state behind a JS-present marker (e.g. a js class set on <html> before observing), so the graphic is visible when JS is absent.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact 
gitar auto-apply:on         

gitar display:verbose

Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source

gitar-bot[bot]

This comment was marked as resolved.

Sign in to view

@offyotto offyotto changed the title new website Update website design Sep 9, 2026
@offyotto offyotto closed this Sep 9, 2026
@offyotto
offyotto deleted the feat/fable-5-website branch September 9, 2026 13:39
@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Sep 9, 2026
@offyotto
offyotto restored the feat/fable-5-website branch September 9, 2026 13:46
@offyotto offyotto reopened this Sep 9, 2026
@offyotto
offyotto dismissed gitar-bot[bot]’s stale review September 9, 2026 13:47

Merging as requested.

@offyotto
offyotto merged commit ab24890 into main Sep 9, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gitar-bot gitar-bot[bot] gitar-bot[bot] left review comments

Assignees

No one assigned

Labels

gitar-approved Added by Gitar

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@offyotto