docs: refresh README screenshots - #25
Conversation
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. |
💡 Codex Reviewhttps://github.com/aspizu/dcim/blob/b139ef28b86bfec65a597a7d05af3bdc770490ea/docs/assets/screenshot0.svg#L8 When these SVGs render in the README, defining https://github.com/aspizu/dcim/blob/b139ef28b86bfec65a597a7d05af3bdc770490ea/docs/assets/screenshot0.svg#L8 When users clone the repository or load the README, the four new SVG blobs total about 8.9 MiB because they inline PNGs as base64, versus roughly 0.69 MiB for all four previous WebP assets. Since the old WebPs also remain in the repository, this documentation refresh permanently adds a disproportionate amount of binary history and makes the rendered README substantially heavier; embedding optimized raster payloads or otherwise preserving compression would avoid that regression. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Refresh the README with the latest gallery, albums, album contents, and HTML embedding screenshots, and move the screenshot section near the top.
Wrap screenshots in self-contained SVGs with rounded corners. Combine the sharing menu and HTML editor screenshots into two cropped panes with a transparent 16px gap, 32px outer corners, and 8px inner corners.
Validation: all four SVGs rendered successfully; the combined crop was visually checked, and pixel checks confirmed transparent corners and the divider. The diff whitespace check passed with the README's existing CRLF line endings accounted for.