Skip to content

Add the promo video to the website and restyle it to match - #104

Merged
gitar-bot[bot] merged 2 commits into
mainfrom
feat/website-promo-video
Sep 28, 2026
Merged

gitar-bot[bot] merged 2 commits into
mainfrom
feat/website-promo-video

Conversation

@offyotto

Copy link
Copy Markdown
Owner

Summary

  • Tour video: a 15-second promo sits right below the hero, in a new "Tour" section with a nav link.
    • docs/videos/core-monitor-tour.mp4: H.264, BT.709, fast start, no audio, 2.2 MB.
    • docs/images/site/core-monitor-tour-poster.jpg: the poster frame.
    • Playback: muted while in view. With reduced motion it waits on the poster. After a manual pause it stays paused.
  • Restyle to match the app and the video:
    • System font (SF Pro / SF Mono on Apple devices).
    • Flat light and dark backgrounds, with no gradients.
    • Rounded cards and pill buttons.
    • MetricTint colors: system blue for actions, thermal orange, cooling teal.
    • The self-hosted Bricolage Grotesque and Spline Sans Mono fonts are removed.
  • Copy cuts:
    • The use-case section ("Where people actually reach for it").
    • The hero and gallery captions.
    • The gallery scroll hint.
    • The compare-table caption.
  • Fixes:
    • 404.html and manifest.webmanifest still used /Core-Monitor/ paths. Those paths return 404 on xnu.is-a.dev, so the 404 page had no styles.
    • The install heading still counted the removed Mac App Store edition.
  • Structured data: adds a VideoObject entry.

Checks

  • Checked in a local preview at desktop and mobile widths, in light and dark mode. No horizontal overflow.
  • The video autoplays muted once it is in view.
  • No background-image gradients are left on the page.
  • The JSON-LD parses, and the HTML tags in index.html and 404.html close correctly.
  • Button text contrast: white on #0071E3 is 4.7:1. Link colors pass AA on both backgrounds.

- Add a 15-second tour video (H.264, BT.709, fast start, 2.2 MB) with a
  poster frame, right below the hero. It plays muted only while in view,
  stays on its poster with reduced motion, and stays paused once a
  visitor pauses it. Add a VideoObject entry to the structured data.
- Restyle the site to the look of the app and the video: system font,
  flat light and dark grounds, rounded cards, and the MetricTint colors.
  Remove the self-hosted Bricolage Grotesque and Spline Sans Mono fonts.
- Remove the use-case section and several secondary captions.
- Fix 404.html and the web manifest, which still pointed at
  /Core-Monitor/ paths that 404 on xnu.is-a.dev.
- Update the install heading, which still counted the removed Mac App
  Store edition.
Comment thread docs/404.html Outdated
gitar-bot[bot]

This comment was marked as resolved.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f757a7da70

ℹ️ 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".

Comment thread docs/404.html Outdated
The /Core-Monitor/ path fix also removed the repository segment from the Download and Releases links, so both returned 404 on GitHub.
@gitar-bot
gitar-bot Bot dismissed their stale review September 28, 2026 01:40

✅ All code review findings resolved.

Configure merge blocking

@gitar-bot
gitar-bot Bot enabled auto-merge (squash) September 28, 2026 01:40

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Sep 28, 2026
@gitar-bot

gitar-bot Bot commented Sep 28, 2026

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

🟡 Medium risk · Adds viewport-triggered video playback and changes the site's installed-app scope.

Adds a 15-second tour video below the hero with muted autoplay and reduced-motion support, alongside a comprehensive restyle to flat backgrounds, system fonts, and MetricTint colors. Resolves the 404 page GitHub links that were dropping the repo name and returning 404 on the custom domain. No issues found.

Auto-approved: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria.

✅ 1 closed
✅ Bug: 404 page GitHub links drop the repo name and lead to a 404

📄 docs/404.html:24 📄 docs/404.html:48
The /Core-Monitor/ path cleanup also removed Core-Monitor/ from two absolute GitHub URLs. The Download button now points to https://github.com/offyotto/releases/latest/download/Core-Monitor.dmg and the footer Releases link to https://github.com/offyotto/releases/latest. Both paths skip the repository segment, so GitHub returns 404. That leaves the 404 page's main download action broken. Put Core-Monitor/ back in both URLs, the same way index.html writes them.

Review coverage

🤖 Auto-approval Approved

📋 Rules No rules evaluated

🧪 Functional validation Not enabled · Set up

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

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
gitar-bot Bot merged commit c035c4a into main Sep 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant