Company Index - Use SSW red for card title and hover border - #5012
Draft
isaaclombardssw wants to merge 2 commits into
Draft
Company Index - Use SSW red for card title and hover border#5012isaaclombardssw wants to merge 2 commits into
isaaclombardssw wants to merge 2 commits into
Conversation
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
|
No linked issues found. Please add the corresponding issues in the pull request description. |
Contributor
Coverage report
Test suite run success35 tests passing in 4 suites. Report generated by 🧪jest coverage report action from 733a372 |
bradystroud
pushed a commit
to SSWConsulting/SSW.Rules.Content
that referenced
this pull request
Sep 3, 2026
Four small follow-ups to the new rule, combined into one PR for review. **Video starts at the right place.** The embed used `youtu.be/...?t=4461`. That parameter only works on the YouTube watch page - embedded players ignore it and start from 0. Switched to `youtube.com/embed/...?start=4461`. **Screenshot-only wording.** Several headings and tips said "screenshot" while the content underneath covers screenshots and Done videos. - "Where the screenshots go" -> "Where the evidence goes" - "Caption every image" -> "Caption every image or video" - "the image ends up in a PR" -> "the capture ends up in a PR" **Real PR screenshots.** The good/bad examples were text mockups of a fictional PR - the last open item from the original PR. Replaced with screenshots of two real draft PRs on SSW.Website ([#5012](SSWConsulting/SSW.Website#5012) and [#5013](SSWConsulting/SSW.Website#5013)), both making the same change: the Company Index card hover border goes from black to SSW red. Captured with Playwright against a local dev server, which is the workflow the rule describes. **Linked the back pressure rule.** [Do you use back pressure to keep AI agents on track?](https://www.ssw.com.au/rules/utilize-back-pressure-for-agents) already describes an agent taking Playwright screenshots and comparing them to the expected state - the same loop, stopping just short of keeping the capture. Added a link in each direction plus reciprocal `related` entries. Frontmatter, check-mdx, malformed-bold and endIntro checks pass. Markdownlint clean. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
isaaclombardssw
changed the title
isaaclombardssw
added a commit
to SSWConsulting/SSW.Rules.Content
that referenced
this pull request
Sep 3, 2026
[#13280](#13280) renamed the rule from `ai-capture-ui-changes` to `done-video-by-ai`. That left three things behind, plus the two changes from my earlier PR that now need to land on the new path. **The old URL 404s.** The rule was live at `/ai-capture-ui-changes`. Added a `redirects` entry for it. **Seven rules linked to the old URI.** `playwright-with-ai`, `definition-of-done`, `done-video`, `write-a-good-pull-request`, `devtools-design-changes`, `ai-assisted-desktop-pr-preview` and `utilize-back-pressure-for-agents` all pointed at `/ai-capture-ui-changes`. Repointed to `/done-video-by-ai`. **Images were left in the old folder.** The rule folder name has to match the uri, so the images moved into `done-video-by-ai/`. **Clearer PR example screenshots.** The demo PR behind the good/bad examples only changed the card hover border, so the pair was hard to read at a glance. It now changes the card title colour too. Recaptured both PR pages, and the images got new names so nothing serves the cached old pair. Demo PRs: [#5012](SSWConsulting/SSW.Website#5012) and [#5013](SSWConsulting/SSW.Website#5013). **Added Puppeteer to the tooling list.** Frontmatter, check-mdx, malformed-bold, endIntro and category-sync all pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5 <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.
Changed the Company Index card title and hover border to SSW red.
🤖 Generated with Claude Code