Skip to content

Add snapshot canvas theme sidecar support#273

Merged
jamieQ merged 1 commit into
mainfrom
feat/canvas-theme-sidecar
Jul 7, 2026
Merged

Add snapshot canvas theme sidecar support#273
jamieQ merged 1 commit into
mainfrom
feat/canvas-theme-sidecar

Conversation

@jamieQ

@jamieQ jamieQ commented Jul 7, 2026

Copy link
Copy Markdown
Member

Adds support for a new metadata property, canvas_theme that can be associated with a snapshot image. When set, the value is respected by the Sentry web UI and renders the image background according to the light or dark value specified. Expose a view modifier and preference that allows clients to control this value, analogous to the way other image metadata is specified.

Refs: EME-1225

@linear-code

linear-code Bot commented Jul 7, 2026

Copy link
Copy Markdown

EME-1225

@cameroncooke cameroncooke left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGMT!

@cameroncooke cameroncooke left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGMT!

@jamieQ jamieQ merged commit 4bd6eed into main Jul 7, 2026
30 checks passed
@jamieQ jamieQ deleted the feat/canvas-theme-sidecar branch July 7, 2026 18:16
mtopo27 pushed a commit to getsentry/sentry-docs that referenced this pull request Jul 9, 2026
## DESCRIBE YOUR PR

Documents the new `canvas_theme` snapshot metadata introduced in
[SnapshotPreviews#273](getsentry/SnapshotPreviews#273).
The field sets the light or dark canvas Sentry renders behind a snapshot
image in the web UI — it's display metadata only and doesn't change the
rendered image.

- Add `canvas_theme` to the platform-agnostic sidecar schema (JSON
example + field table) in `uploading-snapshots`
- Add the `.snapshotCanvasTheme(...)` modifier and `canvas_theme`
sidecar field to the Apple SnapshotPreviews metadata doc (example,
modifier table, sidecar JSON)

## IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs
to go live.
- [ ] Urgent deadline (GA date, etc.): <!-- ENTER DATE HERE -->
- [ ] Other deadline: <!-- ENTER DATE HERE -->
- [x] None: Not urgent, can wait up to 1 week+

## SLA

- Teamwork makes the dream work, so please add a reviewer to your PRs.
- Please give the docs team up to 1 week to review your PR unless you've
added an urgent due date to it.
Thanks in advance for your help!

## PRE-MERGE CHECKLIST

*Make sure you've checked the following before merging your changes:*

- [x] Checked Vercel preview for correctness, including links
- [ ] PR was reviewed and approved by any necessary SMEs (subject matter
experts)
- [ ] PR was reviewed and approved by a member of the [Sentry docs
team](https://github.com/orgs/getsentry/teams/docs)

Co-authored-by: Claude <noreply@anthropic.com>
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.

2 participants