Skip to content

feat: Add full screen support for ObjectPreview iframes#224

Merged
alukach merged 2 commits intosource-cooperative:mainfrom
cmsetzer:feature/support-full-screen-previews
Feb 12, 2026
Merged

feat: Add full screen support for ObjectPreview iframes#224
alukach merged 2 commits intosource-cooperative:mainfrom
cmsetzer:feature/support-full-screen-previews

Conversation

@cmsetzer
Copy link
Copy Markdown
Contributor

What I'm changing

This change enables the ObjectPreview iframe element to launch full screen mode for viewers that request it, such as the image and 3D model viewers from #210.

How I did it

I've added the permissions policy allow="fullscreen" to the iframe in ObjectPreview. This enables any current or future viewer to launch full screen mode.

An alternative approach would be to make allow a custom per-format attribute like style, but this seems simpler.

How you can test it

On a preview branch where this feature is active, navigate to the following pages and click the "Toggle full screen" button within the preview iframe:

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 12, 2026

@alukach is attempting to deploy a commit to the Radiant Earth Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
source-cooperative Ready Ready Preview, Comment Feb 12, 2026 5:15pm

Request Review

@alukach alukach self-requested a review February 12, 2026 17:18
@alukach
Copy link
Copy Markdown
Contributor

alukach commented Feb 12, 2026

So glad you've added this @cmsetzer, I was just looking into how we could do this for #217; really useful addition!

@alukach alukach merged commit 775fc4c into source-cooperative:main Feb 12, 2026
2 checks passed
@cmsetzer cmsetzer deleted the feature/support-full-screen-previews branch February 12, 2026 18:06
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