Skip to content

Add GitHub Pages site and social card - #2

Merged
shadowtor merged 2 commits into
mainfrom
feat/pages-site
Oct 15, 2026
Merged

shadowtor merged 2 commits into
mainfrom
feat/pages-site

Conversation

@shadowtor

Copy link
Copy Markdown
Owner

What this adds

  • site/index.html: a single static page for GitHub Pages. Its centrepiece is a sketch picker that mirrors the skill's flow: three greyscale sketches, and only the one you pick turns to colour and shows its kit (real mockups from examples/). Then how it works, kit contents, merch by business type, install options (Claude Code, other agents, copy-paste prompt), limits and credits.
  • .github/workflows/pages.yml: assembles the page plus the example previews it shows (about 5 MB) and deploys with the official Pages actions.
  • site/social-card.png (1280×640, 69 KB) and tools/make_social_card.py to regenerate it. All content sits inside an 80 px safe area.
  • README links to the site; .gitignore ignores a local _site/ build.

Checked

  • Served locally and driven in a browser: picker works for all three brands (6 mockups load each), no horizontal overflow at 375 px or 1024 px, light and dark schemes, keyboard-focusable radios, reduced-motion respected.
  • Copy checked with SlopMonster: 5/5.

Needed before it goes live

Repo Settings → Pages → Build and deployment → Source: GitHub Actions. After that the workflow publishes to https://shadowtor.github.io/BrandDesign/ on merge.

Social preview for the repo itself is a manual upload (Settings → General → Social preview → site/social-card.png); GitHub has no API for it.

🤖 Generated with Claude Code

- site/index.html: one static page. A sketch picker mirrors the skill's flow
  (greyscale first, colour and kit after you choose), plus how it works, kit
  contents, merch by business type, install options, limits and credits
- .github/workflows/pages.yml: assembles the page and the example previews it
  shows, then deploys with the official Pages actions (needs Pages source set
  to GitHub Actions)
- tools/make_social_card.py and site/social-card.png: social preview at
  1280x640 inside an 80 px safe area (69 KB)
- README links to the site; .gitignore ignores a local _site build

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-15T01:38:28.767604Z 14ea97c Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: 14ea97c628

ℹ️ 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 site/index.html Resolved
@shadowtor

Copy link
Copy Markdown
Owner Author

@codex review

Please focus on: the Pages workflow (permissions, the assemble step, and whether every file the page references is packaged), accessibility and robustness of site/index.html (the sketch picker, the copy buttons, layout at narrow widths), and any broken or wrong links or install commands on the page.

The "Codex, Gemini or another agent" block only showed --codex, which installs
under ~/.codex only. It now lists --codex, --gemini and --dest for other agents.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@shadowtor
shadowtor merged commit 1521db0 into main Oct 15, 2026
3 checks passed
@shadowtor
shadowtor deleted the feat/pages-site branch October 15, 2026 01:37

@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: 14ea97c628

ℹ️ 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 site/index.html
r.addEventListener("change", function () { if (r.checked) show(r.value); });
});
document.querySelectorAll(".cmd").forEach(function (pre) {
var btn = el("button", { "class": "copy", type: "button", "aria-label": "Copy to clipboard" }, "Copy");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Give each copy button a contextual accessible name

For screen-reader users, all three controls have the identical accessible name “Copy to clipboard,” so button-list or quick-navigation users cannot determine whether a control copies the Claude command, installer command, or agent prompt. The fixed aria-label also overrides the changing button text, so the later “Copied” state is not exposed; derive the name from the associated block and expose the result as an announced status.

Useful? React with 👍 / 👎.

Comment thread site/index.html
Comment on lines +338 to +339
if (navigator.clipboard && navigator.clipboard.writeText) { navigator.clipboard.writeText(text).then(done, done); }
else { var t = document.createElement("textarea"); t.value = text; document.body.appendChild(t); t.select(); try { document.execCommand("copy"); } catch (e) {} document.body.removeChild(t); done(); }

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Report clipboard failures instead of success

When navigator.clipboard.writeText() rejects—for example because clipboard permission is denied—the rejection callback is also done, so the button claims “Copied” even though the clipboard remains unchanged. The fallback likewise calls done() after an exception or a false execCommand() result; attempt the fallback on rejection and show success only after a confirmed copy.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant