Skip to content

Show captured photos in Gallery cards - #161

Merged
tiensonqin merged 1 commit into
mainfrom
codex/gallery-photo-previews
Nov 11, 2026
Merged

tiensonqin merged 1 commit into
mainfrom
codex/gallery-photo-previews

Conversation

@tiensonqin

@tiensonqin tiensonqin commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Reproduction

Open a tag with a saved Gallery view and a tagged JPEG asset. The photo record appears as a card with its filename and properties, but no image preview mounts. A regression fixture reproduced this on main.

Changes

  • Carry image asset metadata in compact Gallery results so they can resolve the local file or download it through the existing asset loader.
  • Display a tappable photo preview in the card and retain the title and properties. Document results still render as ordinary cards.

Verification

The tagged Gallery regression passes on both native host contracts and checks that a local capture path reaches the preview. The full OCaml gate and the iOS simulator-target Swift build passed.


Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Devin Review

Comment on lines +532 to +537
let preview = App_test.descendant_with_extension runtime app card "outliner-block-content" in
check ~msg:"the Gallery card mounts the native photo preview" (preview <> -1);
check_eq (App_test.extension_property app preview "asset-type")
(Some (Lui_protocol.StringValue "jpeg"));
check_eq (App_test.extension_property app preview "local-path")
(Some (Lui_protocol.StringValue path));

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Preview test skips image decoding

The fixture never creates the JPEG. The host assertions verify extension properties, not a decoded image or the tap-to-open flow.

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

@tiensonqin
tiensonqin merged commit 9a68baa into main Nov 11, 2026
5 checks passed
@tiensonqin
tiensonqin deleted the codex/gallery-photo-previews branch November 11, 2026 01:19
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