Skip to content

fix(media): never use a scanned document as a portrait - #38

Merged
device25 merged 1 commit into
mainfrom
fix/portrait-photos-only
Oct 25, 2026
Merged

device25 merged 1 commit into
mainfrom
fix/portrait-photos-only

Conversation

@device25

Copy link
Copy Markdown
Owner

People whose first image was a scanned certificate (`FORM.MEDI OTHER`) showed the scan as their round portrait on the chart and in the card.

A portrait is now an image that is a photo (`MEDI PHOTO`) or of unknown medium (most files carry no MEDI). Images with another medium (documents, cards, tombstones) stay in the card's media list and the viewer only. Same rule on the chart (`photoOf`) and in the card (`portraitOf`), via `isPortraitFile`.

🤖 Generated with Claude Code

A person whose first image was a certificate scan (FORM.MEDI OTHER) got
it as the round portrait on the chart and in the card. A portrait is now
an image that is a photo (MEDI PHOTO) or of unknown medium; documents,
cards and tombstones stay in the media list only.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@device25
device25 merged commit 874b794 into main Oct 25, 2026
1 check passed
@device25
device25 deleted the fix/portrait-photos-only branch October 25, 2026 20:27
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