docs: refresh the screenshots, and point the guide at the capture script - #4
Merged
Conversation
The shipped images were stale and low-resolution. dashboard-overview.png was 1440x900 at 1x, showed 0 HIGH RISK and 0 CHANGES THIS WEEK in every stat tile, and predated the "Candidates for clean-up" panel that is now the main dashboard surface — so the one image most readers see first showed an empty-looking app. Recaptured at 2880x1800 (1440x900 @2x) from the canned fixture with a strict privacy profile, so the risk badges carry real values. Added the apps grid and its mobile variant. The bigger fix is the guide. images/SCREENSHOTS.md told contributors to capture manually against a seeded install and never mentioned that `pnpm screenshots` in the app repo does exactly that — seeds the offline fixture, disables the coachmark tour, sets the profile, and writes both themes. The full invocation is now documented, including the two-shell setup and the throwaway data dir. The seed is offline (source=canned uses the in-repo SAMPLE_APPS), so these are reproducible and don't depend on what the App Store is serving that day.
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
2 participants
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.
Two things: the images were stale, and the guide told contributors to do by hand something the app repo automates.
The images
dashboard-overview.pngis the first screenshot most readers see. The shipped version was 1440×900 at 1×, showed0 HIGH RISKand0 CHANGES THIS WEEKin every stat tile, and predates the Candidates for clean-up panel that is now the main dashboard surface. So the headline image made the product look empty.Recaptured at 2880×1800 (1440×900 @2×) from the canned fixture with a strict privacy profile applied, so the risk badges carry real values — six apps triaged across high and moderate. Same for
app-detail-timeline.pngandprivacy-map.png. Addedapps-grid.pngand its mobile variant.images/is 2.5 MB total, inside the 5 MB target the guide sets.The guide
images/SCREENSHOTS.mdsaid to capture manually against a seeded install, and never mentionedpnpm screenshots— which seeds the offline fixture, disables the coachmark tour, sets the profile so badges are non-empty, and writes both light and dark at 2×. Someone built that for exactly this purpose and the docs repo did not know.The full invocation is now documented, including the two-shell setup and the throwaway
PRIVACYTRACKER_DATA_DIR. It covers four of the ten shots in the table; the rest are still manual and marked as such.Worth noting the seed is offline —
source=canneduses the in-repoSAMPLE_APPSfixture and makes no App Store calls — so these are reproducible and do not drift with whatever Apple is serving.Relationship to #3
Independent. Both branch from
mainand touch different files, so they merge in either order. #3 is the text corrections; this is images plus the capture guide.