Skip to content

Commit b3f8218

Browse files
committed
Add plan for PR #1815 .qmd updates (.include-dark, alt text)
Documents the .qmd changes needed in PR #1815 to use the dark mode screenshots: add .include-dark class to image references, update alt text from Twitter to Bluesky.
1 parent 9841fd2 commit b3f8218

2 files changed

Lines changed: 76 additions & 0 deletions

File tree

.claude/plans/screenshot/00-index.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@ Claude Code is the primary operator. Scripts handle deterministic work (zero AI
2828
| 05 | [05-documentation.md](05-documentation.md) | CLAUDE.md, SETUP.md, visual rules | Revised |
2929
| 06 | [06-skill-scripts.md](06-skill-scripts.md) | `/screenshot` skill, `!` preprocessing, helper scripts | New |
3030
| 07 | [07-capture-agent.md](07-capture-agent.md) | `screenshot-capture` agent definition | New |
31+
| 08 | [08-walkthrough-learnings.md](08-walkthrough-learnings.md) | Learnings from Phase 0.5 walkthrough | Updated |
32+
| 09 | [09-pr-1815-screenshot-updates.md](09-pr-1815-screenshot-updates.md) | .qmd changes for PR #1815 (`.include-dark`, alt text) | New |
3133

3234
## Key Decisions
3335

Lines changed: 74 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,74 @@
1+
# Sub-plan 09: PR #1815 Screenshot Updates
2+
3+
Parent: [00-index.md](00-index.md)
4+
PR: https://github.com/quarto-dev/quarto-web/pull/1815
5+
6+
## Context
7+
8+
PR #1815 moves examples from Twitter to Bluesky across quarto-web docs. Several doc pages reference screenshots that need updating with the new icons and examples. This plan describes what to do in that PR using the screenshot tooling built in this branch.
9+
10+
## Screenshots to Update
11+
12+
The tooling branch produces these images via `capture.js`:
13+
14+
| Image | Light | Dark | Dimensions |
15+
|-------|-------|------|------------|
16+
| about-jolla | `about-jolla.png` | `about-jolla-dark.png` | 819x579 |
17+
| navbar-tools | `navbar-tools.png` | `navbar-tools-dark.png` | 181x136 |
18+
19+
## .qmd Changes Needed in PR #1815
20+
21+
Add `.include-dark` class to image references so the `filters/include-dark.lua` filter generates both light and dark `<img>` tags. The filter expects a `-dark` variant at the same path (e.g., `about-jolla.png` + `about-jolla-dark.png`).
22+
23+
### `docs/websites/website-about.qmd` (line 69)
24+
25+
```markdown
26+
# Before
27+
![](images/about-jolla.png){.border fig-alt="..."}
28+
29+
# After
30+
![](images/about-jolla.png){.border .include-dark fig-alt="..."}
31+
```
32+
33+
### `docs/websites/website-blog.qmd` (line 168)
34+
35+
```markdown
36+
# Before
37+
![](images/about-jolla.png){.border fig-alt="..."}
38+
39+
# After
40+
![](images/about-jolla.png){.border .include-dark fig-alt="..."}
41+
```
42+
43+
### `docs/websites/website-navigation.qmd` (line 112)
44+
45+
```markdown
46+
# Before
47+
![](images/navbar-tools.png){.border alt="...Twitter..."}
48+
49+
# After
50+
![](images/navbar-tools.png){.border .include-dark alt="...Bluesky..."}
51+
```
52+
53+
Note: the navbar-tools alt text should also be updated from "Twitter" to "Bluesky" since the icon changed.
54+
55+
## Alt Text Updates
56+
57+
The example projects now use Bluesky instead of Twitter. Update alt text in the .qmd files:
58+
- "Twitter and Github logo" → "Bluesky and Github logo"
59+
- "buttons for twitter and github" → "buttons for Bluesky and GitHub" (or similar)
60+
61+
## Workflow
62+
63+
1. Merge the screenshot tooling branch first (or cherry-pick the images)
64+
2. In PR #1815, add `.include-dark` to the three image references above
65+
3. Copy the four screenshot files (light + dark for both) into the PR
66+
4. Update alt text to reference Bluesky
67+
5. Verify with `quarto preview` that both light and dark images render correctly
68+
69+
## Future Screenshots
70+
71+
When more screenshots are added to the manifest (sidebar, blog, etc.), follow the same pattern:
72+
- Set `"dark": true` in manifest.json
73+
- Add `.include-dark` to the .qmd image reference
74+
- Both variants are captured automatically by `capture.js`

0 commit comments

Comments
 (0)