Skip to content

Refine figure embed URL handling and fix regression for http(s) figure images - #1290

Open
cbutcosk wants to merge 9 commits into
fix/figures-media-video-audio-postersfrom
fix/figures-media-embed-data
Open

Refine figure embed URL handling and fix regression for http(s) figure images#1290
cbutcosk wants to merge 9 commits into
fix/figures-media-video-audio-postersfrom
fix/figures-media-embed-data

Conversation

@cbutcosk

@cbutcosk cbutcosk commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

This PR refactors the figures plugin to normalize URL calculations for embed figures (youtube, vimeo, soundcloud) and fix a regression in http(s) figures (see #1289). It is the final part of #1197 . Details:

  • Adds a test ensuring that figure embed URLs are generated on the figureMedia object and are properly formatted.
  • Adds a test ensuring that figure images with http(s) URLs download the image URL for use in dimension calculations.
  • Moves figure media embed calculations into FigureMedia processing loop, inserting embedUrl and sourceUrl on derivatives.embed. Refactors figureMediaEmbedUrl component to be a helper function of the figures plugin.
  • Adds a code path to FigureMedia that fetches http(s) URLs for use in dimensions calculation. Removes component-side figure embed calculation.

@cbutcosk
cbutcosk changed the base branch from main to fix/figures-media-video-audio-posters August 31, 2026 13:28
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.

Figure images with http(s) sources generate an error with @thegetty/quire-11ty@1.0.1-rc.1 Transform global data (figure.src)

1 participant