Skip to content

docs: refresh README screenshots - #25

Merged
aspizu merged 1 commit into
mainfrom
apz/readme-screenshots
Sep 14, 2026
Merged

aspizu merged 1 commit into
mainfrom
apz/readme-screenshots

Conversation

@aspizu

@aspizu aspizu commented Sep 14, 2026

Copy link
Copy Markdown
Owner

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.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-14T21:03:59.247765Z b139ef2 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@aspizu
aspizu merged commit ca0afd8 into main Sep 14, 2026
5 checks passed
@aspizu
aspizu deleted the apz/readme-screenshots branch September 14, 2026 20:59
@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

https://github.com/aspizu/dcim/blob/b139ef28b86bfec65a597a7d05af3bdc770490ea/docs/assets/screenshot0.svg#L8
P2 Badge Apply the clipping path to each screenshot

When these SVGs render in the README, defining clipPath#corners has no effect unless it is referenced by the <image> or a containing group. The embedded PNG is opaque at its corners, so this screenshot remains square rather than receiving the intended 32px rounded corners; the same omission appears on line 8 of screenshot1.svg and screenshot2.svg, while screenshot3.svg demonstrates the required clip-path usage.


https://github.com/aspizu/dcim/blob/b139ef28b86bfec65a597a7d05af3bdc770490ea/docs/assets/screenshot0.svg#L8
P2 Badge Reduce the embedded screenshot payloads

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 GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant