Skip to content

Use desktop screenshots for conversation reassignment - #408

Merged
pelcasandra merged 1 commit into
mainfrom
codex/assigning-conversations-desktop-figures
Oct 11, 2026
Merged

pelcasandra merged 1 commit into
mainfrom
codex/assigning-conversations-desktop-figures

Conversation

@pelcasandra

Copy link
Copy Markdown
Member

Summary

Fix the desktop presentation reported in the assigning-conversations guide. The teammate menu and reassignment result previously used mobile screenshots as their default images, including on desktop.

  • Use genuine retained desktop sources for those two figures in Spanish and English.
  • Add four lossless focused derivatives with their original 4× density and Display P3 ICC profile preserved.
  • Keep the existing mobile picture sources for screens up to 600px.
  • Record original provenance, exact crop coordinates, pixel hashes and complete browser review.

The article prose, titles, slugs, links and publication metadata are unchanged. This is a focused correction to the published guide.

Verification

  • Ruby 3.3.6 production build and security header verification passed.
  • All 24 selected PNGs match the root and Spanish build exports exactly.
  • Complete ES/EN browser review at 1440×1000, 580×900 and 390×844 through the footer; all 70 diagnostic screenshots inspected.
  • No page overflow, missing images or source upscaling; both complete desktop menus and assignment record are readable alongside their instructions.
  • Original desktop source pixels and ICC match the exact retained parent subregions.
  • git diff --check passed.

Source and local verification records are in docs/editorial-work/captures/assigning-desktop-correction-20261011/. Normal deployment and public revision checks will be verified separately after merge.

@netlify

netlify Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for legendary-lollipop-e2d131 ready!

Name Link
🔨 Latest commit 19a54ba
🔍 Latest deploy log https://app.netlify.com/projects/legendary-lollipop-e2d131/deploys/6acbdc00e46b8000084caa86
😎 Deploy Preview https://deploy-preview-408--legendary-lollipop-e2d131.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 11, 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-10-11T19:02:20.279189Z 19a54ba Manual request
ℹ️ 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.

@pelcasandra

Copy link
Copy Markdown
Member Author

@codex review

Please review exact head 19a54ba1b8c68b558ece917762ae85993ff2c284. This focused ES/EN correction replaces two mobile default images with genuine desktop crops while preserving the mobile picture sources. Check image density and dimensions, the original-source provenance, and unintended article or publication changes. The production build and complete six-view browser QA passed locally.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 19a54ba1b8

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

@pelcasandra
pelcasandra merged commit 7cdf847 into main Oct 11, 2026
6 checks passed
@pelcasandra

Copy link
Copy Markdown
Member Author

Production verification

Merged normally as 7cdf84707d38f8f155b189d852d29501a7833c10; both original parents remain in history.

Main Build 38166502428 passed for that exact merge commit.
Normal Netlify production deploy 6acbddd97e395000085ea4f3 is ready and published with the matching commit_ref at 2026-10-11T19:05:40.010Z. No manual deploy was used.

Both public HTML pages and all 24 selected PNGs returned HTTP 200 and matched the reviewed local build byte for byte. The changed desktop menu and reassignment result were also inspected in the public browser in English and Spanish. The full local six-view review remains recorded in the committed local verification.

Public pages

Public image verification

Four exact rectangular desktop derivatives were added; original source parents and existing mobile variants were reused unchanged. No source app, account, fixture, or assignment state was created for this correction.

PNG SHA-256
ai-chooser-en-desktop.png 3414bc056e4d3b10adb54b6912c6f982c63e4f1cb8486cbebd933bcf6131bdc6
ai-chooser-en-mobile.png 3a0ddb66c2b580b97c5276c4c2f0cdb706584c6502d809b96858e674eba9da37
ai-chooser-es-desktop.png 5eafea8aa8328a4c99838b86e56a93d882a5682360642df5d7f4310e664b9a4a
ai-chooser-es-mobile.png bdc04cd34104df2a4c17c9d2fb2c01ca9927615c34f793321266921ed1393c2e
bulk-chooser-en-desktop.png 018ea1d8a0e77e68ad5fb918c8f11f8cc449643b1f83b7259ec4e18ce07a7d3d
bulk-chooser-en-mobile.png 8bdec2cb3c3b8a5d5728300012d17c7eaf562d04e3455e8b0d81b589d3f8fa52
bulk-chooser-es-desktop.png f104c2a1ffa4fe874b6e3295ecd117d543cbde426a9a7bf7d99476d24984fa8c
bulk-chooser-es-mobile.png 15926d24ee07b202c0e01287e222f3cee9d22f042dc548d29c4d5dcda515001f
claim-after-en-desktop.png 6340c8cbad83c9f260a5e205fea9779d85db868a11998d9f860a18512ad019a9
claim-after-en-mobile.png 6c96b4e5f59a2830df6309d412134f14a13bbed9475cb753e45c757befbbefa0
claim-after-es-desktop.png 065488c0be824de7ce7cb520ada8f40acc014620bdf1826fe88cc4a8a576cd65
claim-after-es-mobile.png f87c834cb89388fa41c2ed9c978edf1a8ee80eff049c6099451c471b90eaf204
claim-before-en-desktop.png d3d562784199a8bbb3884b0be2ad88693476b777ebdf743b9bdf28068ff5aa99
claim-before-en-mobile.png 6ad88b1405c5b96cefebc9858782a64ef3cb2cab89704b2d4ccbdbf1ba2d6ad5
claim-before-es-desktop.png 70fa655f7a5a0318e1cd7816b5bfc0bd4fa33fb64fa71c80377fc955904c8120
claim-before-es-mobile.png c0f427f0fb9d7fc3134663adbf4174261a001ea0f0a8ca2a0915e0b0a388009c
assignment-menu-en-desktop.png 89d5a7a7e4d494656dfd37843058e1748dd21018251db9503f2e15cc0b1a9f68
assignment-menu-es-desktop.png eb440b673236a86396821b28afe0807124a2fad7c4352535405c08f0beb04adb
assignment-result-en-desktop.png e6a1c9ed05b067e67d4c166ad34912d210a71b1e37515723d0ad45df37ec8eff
assignment-result-es-desktop.png d251907eb1927039396aebb88fe6c8a8ad2a75312feb1f7677165b7b4a43d536
assignment-menu-en-mobile.png ce94aaf4b87afe7d6d210db10d16defef9d8f354d2531296b3ae6ca0c02edbf1
assignment-menu-es-mobile.png f776e121085bac8d2f02c9af32e9f4a61100e6589d256e4344ee222b9700dd0c
assignment-result-en-mobile.png d5aa61706023b820170221d098d991a8df8fddc07d2c657cc037d6927c27ee7f
assignment-result-es-mobile.png fea95c1b1df5b9df14b249a3e0128b5281a71675f916e5e6e8ff96288390d31b

The independent review completed on exact head 19a54ba1b8c68b558ece917762ae85993ff2c284 with no findings. All comments, reviews, and review threads were read before merging; no unresolved threads were present.

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