Skip to content

Convert container-only SVGs to AVIF - #5452

Merged
axelhzf merged 1 commit into
mainfrom
fix-svg-with-embedded-images
Jul 31, 2026
Merged

Convert container-only SVGs to AVIF#5452
axelhzf merged 1 commit into
mainfrom
fix-svg-with-embedded-images

Conversation

@axelhzf

@axelhzf axelhzf commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

While working on #5451, we discovered several SVG files in public/ that were just containers wrapping a single embedded raster image with no meaningful vector content. Extract the embedded images and convert them to AVIF (quality 80) for smaller file sizes, and update all references across the docs.

While working on #5451, we discovered several SVG
files in public/ that were just containers wrapping a single embedded
raster image with no meaningful vector content. Extract the embedded
images and convert them to AVIF (quality 80) for smaller file sizes,
and update all references across the docs.

Images with a background rect behind the embedded image are rasterized
directly from the full SVG (via rsvg-convert) rather than extracting
the raw embedded bytes, to preserve the background color.
@axelhzf
axelhzf requested review from a team July 30, 2026 11:41
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot

Copy link
Copy Markdown
Contributor

One or more of the following people are relevant to this code:

@Alienah Alienah left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It looks good to me!

@abbycross abbycross left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!!

@axelhzf
axelhzf added this pull request to the merge queue Jul 31, 2026
Merged via the queue into main with commit dd74ad0 Jul 31, 2026
5 of 6 checks passed
@axelhzf
axelhzf deleted the fix-svg-with-embedded-images branch July 31, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants