docs: retake outdated screenshots and demo GIFs - #1822
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-01a0deb1-657e-701c-a05a-98672ce4f276 Co-authored-by: Christian Bager Bach Houmann <christian@bagerbach.com>
…hots after #1809 Amp-Thread-ID: https://ampcode.com/threads/T-01a0deb1-657e-701c-a05a-98672ce4f276 Co-authored-by: Christian Bager Bach Houmann <christian@bagerbach.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (30)
📒 Files selected for processing (10)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 4 remain after this review. 📝 WalkthroughWalkthroughDocumentation updates revise setup and choice instructions, clarify macro, capture, and template examples, and update screenshots, captions, and an image reference. ChangesDocumentation updates
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to No actionable merge risk is established in the reviewed documentation updates, and the Toggl instructions match the supported Macro-choice workflow. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit reads the setup page, Comment |
Deploying quickadd with
|
| Latest commit: |
781b120
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://690fc88f.quickadd.pages.dev |
| Branch Preview URL: | https://docs-refresh-screenshots.quickadd.pages.dev |
Summary
Retakes the outdated screenshots and demo GIFs in the docs, from real Obsidian 1.13.7 with the current build and a throwaway demo vault. Each choice was set up through the UI and run, so the images match the page text.
docs/public/img/choices/*,docs/public/img/examples/*, theissue-1642-*images,TogglManagerMacroChoice.png, and the old.jpgmovie screenshot.Text changes
Verification
pnpm run buildindocs/passes (52 pages); every new image appears in the built HTML, and the GIFs stay animated after Astro's WebP conversion.Images/link in the docs resolves to an existing file.Previews: Template builder · Macro builder · Multi list · AI setup GIF · AI macro GIF · Todoist GIF
Release / migration impact
Docs only. No plugin code,
main.js,manifest.jsonorversions.jsonchanges.Follow-up
The AI Setup GIF shows the provider editor from before #1818. I'll re-record it once the provider editor moves to a settings sub-page (item 6), since that changes the flow again.
Note
Refresh outdated screenshots and demo GIFs in docs
docs/src/content/docs/docs/ImagesAI promptsfolder setup in AIAssistant.md, the row drag handle in MultiChoice.md, and the TogglManager user script flow in Macro_TogglManager.mdmoviescript_settings.jpg,TogglManagerMacroChoice.png, and all images underdocs/public/imgMacroscope summarized 781b120.
Summary by CodeRabbit