Merge f9862eba8d8bd2ed239240464a1a7721aacf96e7 into b790c6d59ceaf7a809cc24b60c1f1509807470e2

.github/workflows/ci.yml

@@ -40,7 +40,7 @@ jobs:
           sudo dpkg --add-architecture i386
           sudo apt update
           sudo apt install -y gcc-multilib libssl-dev:i386 pkg-config:i386
-      - uses: dtolnay/rust-toolchain@1.87.0
+      - uses: dtolnay/rust-toolchain@1.88.0
         with:
           targets: ${{ matrix.bits == 32 && 'i686-unknown-linux-gnu' || '' }}
       - uses: Swatinem/rust-cache@v2
@@ -73,7 +73,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: dtolnay/rust-toolchain@1.87.0
+      - uses: dtolnay/rust-toolchain@1.88.0
         with:
           components: clippy, rustfmt
       - uses: Swatinem/rust-cache@v2

Cargo.lock

@@ -415,15 +415,6 @@ name = "codex"
 version = "0.1.1"
 source = "git+https://github.com/typst/codex?rev=9ac86f9#9ac86f96af5b89fce555e6bba8b6d1ac7b44ef00"
 
-[[package]]
-name = "color"
-version = "0.3.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7ae467d04a8a8aea5d9a49018a6ade2e4221d92968e8ce55a48c0b1164e5f698"
-dependencies = [
- "bytemuck",
-]
-
 [[package]]
 name = "color-print"
 version = "0.3.7"
@@ -973,9 +964,23 @@ dependencies = [
  "url",
 ]
 
+[[package]]
+name = "hayro"
+version = "0.1.0"
+source = "git+https://github.com/LaurenzV/hayro?rev=d385360#d38536089c95b521b0b64080302d7a305344edbf"
+dependencies = [
+ "bytemuck",
+ "hayro-interpret",
+ "image",
+ "kurbo",
+ "rustc-hash",
+ "smallvec",
+]
+
 [[package]]
 name = "hayro-font"
 version = "0.1.0"
+source = "git+https://github.com/LaurenzV/hayro?rev=d385360#d38536089c95b521b0b64080302d7a305344edbf"
 dependencies = [
  "log",
  "phf",
@@ -984,13 +989,13 @@ dependencies = [
 [[package]]
 name = "hayro-interpret"
 version = "0.1.0"
+source = "git+https://github.com/LaurenzV/hayro?rev=d385360#d38536089c95b521b0b64080302d7a305344edbf"
 dependencies = [
  "bitflags 2.9.1",
  "hayro-font",
  "hayro-syntax",
  "kurbo",
  "log",
- "peniko",
  "phf",
  "qcms",
  "skrifa",
@@ -998,23 +1003,10 @@ dependencies = [
  "yoke 0.8.0",
 ]
 
-[[package]]
-name = "hayro-render"
-version = "0.1.0"
-dependencies = [
- "bytemuck",
- "hayro-interpret",
- "hayro-syntax",
- "image",
- "kurbo",
- "peniko",
- "rustc-hash",
- "smallvec",
-]
-
 [[package]]
 name = "hayro-syntax"
 version = "0.0.1"
+source = "git+https://github.com/LaurenzV/hayro?rev=d385360#d38536089c95b521b0b64080302d7a305344edbf"
 dependencies = [
  "flate2",
  "kurbo",
@@ -1027,6 +1019,7 @@ dependencies = [
 [[package]]
 name = "hayro-write"
 version = "0.1.0"
+source = "git+https://github.com/LaurenzV/hayro?rev=d385360#d38536089c95b521b0b64080302d7a305344edbf"
 dependencies = [
  "flate2",
  "hayro-syntax",
@@ -1276,9 +1269,9 @@ checksum = "cb56e1aa765b4b4f3aadfab769793b7087bb03a4ea4920644a6d238e2df5b9ed"
 
 [[package]]
 name = "image"
-version = "0.25.5"
+version = "0.25.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cd6f44aed642f18953a158afeb30206f4d50da59fbc66ecb53c66488de73563b"
+checksum = "db35664ce6b9810857a38a906215e75a9c879f0696556a39f59c62829710251a"
 dependencies = [
  "bytemuck",
  "byteorder-lite",
@@ -1437,6 +1430,7 @@ dependencies = [
 [[package]]
 name = "krilla"
 version = "0.4.0"
+source = "git+https://github.com/LaurenzV/krilla?rev=f7d753c3#f7d753c39e1fb7118fb1b774243a0720771f229f"
 dependencies = [
  "base64",
  "bumpalo",
@@ -1466,6 +1460,7 @@ dependencies = [
 [[package]]
 name = "krilla-svg"
 version = "0.1.0"
+source = "git+https://github.com/LaurenzV/krilla?rev=f7d753c3#f7d753c39e1fb7118fb1b774243a0720771f229f"
 dependencies = [
  "flate2",
  "fontdb",
@@ -1923,18 +1918,6 @@ dependencies = [
  "ryu",
 ]
 
-[[package]]
-name = "peniko"
-version = "0.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1f9529efd019889b2a205193c14ffb6e2839b54ed9d2720674f10f4b04d87ac9"
-dependencies = [
- "bytemuck",
- "color",
- "kurbo",
- "smallvec",
-]
-
 [[package]]
 name = "percent-encoding"
 version = "2.3.1"
@@ -3256,7 +3239,7 @@ version = "0.13.1"
 dependencies = [
  "bytemuck",
  "comemo",
- "hayro-render",
+ "hayro",
  "image",
  "pixglyph",
  "resvg",
@@ -3275,6 +3258,7 @@ dependencies = [
  "comemo",
  "ecow",
  "flate2",
+ "hayro",
  "image",
  "ttf-parser",
  "typst-library",

Cargo.toml

@@ -61,8 +61,8 @@ fontdb = { version = "0.23", default-features = false }
 fs_extra = "1.3"
 glidesort = "0.1.2"
 hayagriva = "0.8.1"
-hayro-syntax = { path = "../hayro/hayro-syntax" }
-hayro-render = { path = "../hayro/hayro-render" }
+hayro-syntax = { git = "https://github.com/LaurenzV/hayro", rev = "d385360" }
+hayro = { git = "https://github.com/LaurenzV/hayro", rev = "d385360" }
 heck = "0.5"
 hypher = "0.1.4"
 icu_properties = { version = "1.4", features = ["serde"] }
@@ -75,8 +75,8 @@ image = { version = "0.25.5", default-features = false, features = ["png", "jpeg
 indexmap = { version = "2", features = ["serde"] }
 infer = { version = "0.19.0", default-features = false }
 kamadak-exif = "0.6"
-krilla = { path = "../krilla/crates/krilla", default-features = false, features = ["raster-images", "comemo", "rayon", "pdf"] }
-krilla-svg = { path = "../krilla/crates/krilla-svg" }
+krilla = { git = "https://github.com/LaurenzV/krilla", rev = "f7d753c3", default-features = false, features = ["raster-images", "comemo", "rayon", "pdf"] }
+krilla-svg = { git = "https://github.com/LaurenzV/krilla", rev = "f7d753c3" }
 kurbo = "0.11"
 libfuzzer-sys = "0.4"
 lipsum = "0.9"

crates/typst-library/src/visualize/image/mod.rs

@@ -4,6 +4,7 @@ mod pdf;
 mod raster;
 mod svg;
 
+pub use self::pdf::PdfImage;
 pub use self::raster::{
     ExchangeFormat, PixelEncoding, PixelFormat, RasterFormat, RasterImage,
 };
@@ -14,10 +15,11 @@ use std::fmt::{self, Debug, Formatter};
 use std::sync::Arc;
 
 use ecow::EcoString;
+use typst_library::{Feature, World};
 use typst_syntax::{Span, Spanned};
 use typst_utils::LazyHash;
 
-use crate::diag::{warning, At, LoadedWithin, SourceResult, StrResult};
+use crate::diag::{bail, warning, At, LoadedWithin, SourceResult, StrResult};
 use crate::engine::Engine;
 use crate::foundations::{
     cast, elem, func, scope, Bytes, Cast, Content, Derived, NativeElement, Packed, Smart,
@@ -27,7 +29,7 @@ use crate::layout::{Length, Rel, Sizing};
 use crate::loading::{DataSource, Load, LoadSource, Loaded, Readable};
 use crate::model::Figurable;
 use crate::text::{families, LocalName};
-use crate::visualize::image::pdf::{PdfDocument, PdfImage};
+use crate::visualize::image::pdf::PdfDocument;
 
 /// A raster or vector graphic.
 ///
@@ -269,13 +271,42 @@ impl Packed<ImageElem> {
                 .within(loaded)?,
             ),
             ImageFormat::Vector(VectorFormat::Pdf) => {
-                let document = PdfDocument::new(loaded.data.clone()).within(loaded)?;
-                // The user provides the page number staring from 1, further down the pipeline they page
-                // numbers are 0-based.
-                let pdf_image =
-                    PdfImage::new(document, self.page.get(styles) - 1).within(loaded)?;
+                if engine.world.library().features.is_enabled(Feature::PdfEmbedding) {
+                    let document =
+                        PdfDocument::new(loaded.data.clone(), engine.world.clone())
+                            .within(loaded)?;
+                    let page_num = self.page.get(styles);
 
-                ImageKind::Pdf(pdf_image)
+                    if page_num == 0 {
+                        bail!(
+                            span,
+                            "{page_num} is not a valid page number";
+                            hint: "page numbers for PDF start at 1"
+                        )
+                    };
+
+                    // The user provides the page number start from 1, further down the pipeline,
+                    // page numbers are 0-based.
+                    let page_idx = page_num - 1;
+                    let num_pages = document.len();
+
+                    let Some(pdf_image) = PdfImage::new(document, page_idx) else {
+                        bail!(
+                            span,
+                            "page {page_num} doesn't exist";
+                            hint: "the document only has {num_pages} pages"
+                        );
+                    };
+
+                    ImageKind::Pdf(pdf_image)
+                } else {
+                    bail!(
+                        span,
+                        "embedding PDFs is currently an experimental, opt-in feature";
+                        hint: "enable the corresponding feature to try it out";
+                        hint: "convert your PDF to SVG instead"
+                    );
+                }
             }
         };
 

crates/typst-library/src/visualize/image/pdf.rs

@@ -1,15 +1,17 @@
 use crate::diag::LoadResult;
 use crate::foundations::Bytes;
-use hayro_syntax::pdf::Pdf;
+use crate::text::{FontStretch, FontStyle, FontVariant, FontWeight};
+use crate::World;
+use comemo::Tracked;
+use hayro_syntax::page::Page;
+use hayro_syntax::Pdf;
 use std::hash::{Hash, Hasher};
 use std::sync::Arc;
-use hayro_syntax::document::page::Page;
 
-#[derive(Clone)]
 struct DocumentRepr {
     pdf: Arc<Pdf>,
     data: Bytes,
-    page_sizes: Vec<(f32, f32)>,
+    standard_fonts: Arc<StandardFonts>,
 }
 
 impl Hash for DocumentRepr {
@@ -26,14 +28,16 @@ impl PdfDocument {
     /// Load a PDF document.
     #[comemo::memoize]
     #[typst_macros::time(name = "load pdf document")]
-    pub fn new(data: Bytes) -> LoadResult<PdfDocument> {
+    pub fn new(data: Bytes, world: Tracked<dyn World + '_>) -> LoadResult<PdfDocument> {
         // TODO: Remove unwraps
         let pdf = Arc::new(Pdf::new(Arc::new(data.clone())).unwrap());
-        let pages = pdf.pages().unwrap();
+        let standard_fonts = get_standard_fonts(world.clone());
 
-        let page_sizes = pages.get().iter().map(|p| p.render_dimensions()).collect();
+        Ok(Self(Arc::new(DocumentRepr { data, pdf, standard_fonts })))
+    }
 
-        Ok(Self(Arc::new(DocumentRepr { data, pdf, page_sizes })))
+    pub fn len(&self) -> usize {
+        self.0.pdf.pages().len()
     }
 }
 
@@ -56,13 +60,14 @@ impl Hash for ImageRepr {
 pub struct PdfImage(Arc<ImageRepr>);
 
 impl PdfImage {
+    /// Create a new PDF image. Returns `None` if the page index is not valid.
     #[comemo::memoize]
-    pub fn new(document: PdfDocument, page: usize) -> LoadResult<PdfImage> {
+    pub fn new(document: PdfDocument, page: usize) -> Option<PdfImage> {
         // TODO: Don't allow loading if pdf-embedding feature is disabled.
         // TODO: Remove Unwrap
-        let dimensions = *(&document.0).page_sizes.get(page).unwrap();
+        let dimensions = document.0.pdf.pages().get(page)?.render_dimensions();
 
-        Ok(Self(Arc::new(ImageRepr {
+        Some(Self(Arc::new(ImageRepr {
             document,
             page_index: page,
             width: dimensions.0,
@@ -70,18 +75,22 @@ impl PdfImage {
         })))
     }
 
-    pub fn with_page<F, R>(&self, f: F) -> R
-    where
-        F: FnOnce(&Page) -> R,
-    {
-        let pages = self.0.document.0.pdf.pages().unwrap();
-        f(&pages.get().get(self.0.page_index).unwrap())
+    pub fn page(&self) -> &Page {
+        &self.0.document.0.pdf.pages()[self.0.page_index]
+    }
+
+    pub fn pdf(&self) -> &Arc<Pdf> {
+        &self.0.document.0.pdf
     }
 
     pub fn width(&self) -> f32 {
         self.0.width
     }
 
+    pub fn standard_fonts(&self) -> &Arc<StandardFonts> {
+        &self.0.document.0.standard_fonts
+    }
+
     pub fn height(&self) -> f32 {
         self.0.height
     }
@@ -93,4 +102,83 @@ impl PdfImage {
     pub fn page_index(&self) -> usize {
         self.0.page_index
     }
+
+    pub fn document(&self) -> &PdfDocument {
+        &self.0.document
+    }
+}
+
+#[comemo::memoize]
+fn get_standard_fonts(world: Tracked<dyn World + '_>) -> Arc<StandardFonts> {
+    let book = world.book();
+
+    let get_font = |name: &str, fallback_name: Option<&str>, variant: FontVariant| {
+        book.select(name, variant)
+            .or_else(|| {
+                if let Some(fallback_name) = fallback_name {
+                    book.select(fallback_name, variant)
+                } else {
+                    None
+                }
+            })
+            .and_then(|i| world.font(i))
+            .map(|font| font.data().clone())
+    };
+
+    let normal_variant = FontVariant::new(
+        FontStyle::Normal,
+        FontWeight::default(),
+        FontStretch::default(),
+    );
+    let bold_variant =
+        FontVariant::new(FontStyle::Normal, FontWeight::BOLD, FontStretch::default());
+    let italic_variant = FontVariant::new(
+        FontStyle::Italic,
+        FontWeight::default(),
+        FontStretch::default(),
+    );
+    let bold_italic_variant =
+        FontVariant::new(FontStyle::Italic, FontWeight::BOLD, FontStretch::default());
+
+    let helvetica = VariantFont {
+        normal: get_font("helvetica", Some("liberation sans"), normal_variant),
+        bold: get_font("helvetica", Some("liberation sans"), bold_variant),
+        italic: get_font("helvetica", Some("liberation sans"), italic_variant),
+        bold_italic: get_font("helvetica", Some("liberation sans"), bold_italic_variant),
+    };
+
+    let courier = VariantFont {
+        normal: get_font("courier", Some("liberation mono"), normal_variant),
+        bold: get_font("courier", Some("liberation mono"), bold_variant),
+        italic: get_font("courier", Some("liberation mono"), italic_variant),
+        bold_italic: get_font("courier", Some("liberation mono"), bold_italic_variant),
+    };
+
+    let times = VariantFont {
+        normal: get_font("times", Some("liberation serif"), normal_variant),
+        bold: get_font("times", Some("liberation serif"), bold_variant),
+        italic: get_font("times", Some("liberation serif"), italic_variant),
+        bold_italic: get_font("times", Some("liberation serif"), bold_italic_variant),
+    };
+
+    // TODO: Use Foxit fonts as fallback
+    let symbol = get_font("symbol", None, normal_variant);
+    let zapf_dingbats = get_font("zapf dingbats", None, normal_variant);
+
+    Arc::new(StandardFonts { helvetica, courier, times, symbol, zapf_dingbats })
+}
+
+pub struct VariantFont {
+    pub normal: Option<Bytes>,
+    pub bold: Option<Bytes>,
+    pub italic: Option<Bytes>,
+    pub bold_italic: Option<Bytes>,
+}
+
+pub struct StandardFonts {
+    pub helvetica: VariantFont,
+    pub courier: VariantFont,
+    pub times: VariantFont,
+    pub symbol: Option<Bytes>,
+    pub zapf_dingbats: Option<Bytes>,
 }

crates/typst-pdf/src/convert.rs

@@ -9,6 +9,7 @@ use krilla::embed::EmbedError;
 use krilla::error::KrillaError;
 use krilla::geom::PathBuilder;
 use krilla::page::{PageLabel, PageSettings};
+use krilla::pdf::PdfError;
 use krilla::surface::Surface;
 use krilla::{Document, SerializeSettings};
 use krilla_svg::render_svg_glyph;
@@ -364,9 +365,37 @@ fn finish(
                 )
             }
             KrillaError::Pdf(_, e, loc) => {
-                // TODO: Better errors
                 let span = to_span(loc);
-                bail!(span, "failed to process PDF");
+                match e {
+                    // We already validated in `typst-library` that the page index is valid.
+                    PdfError::InvalidPage(_) => unreachable!(),
+                    PdfError::VersionMismatch(v) => {
+                        let pdf_ver = v.as_str();
+                        let config_ver = configuration.version();
+                        let cur_ver = config_ver.as_str();
+
+                        bail!(span,
+                            "the version of the PDF file is too high";
+                            hint: "the current export target is {cur_ver}, while the PDF has version {pdf_ver}";
+                            hint: "raise the export target to {pdf_ver} or higher";
+                            hint: "preprocess the PDF to convert it to a lower version"
+                        );
+                    }
+                }
+            }
+            KrillaError::DuplicateTagId(_, loc) => {
+                let span = to_span(loc);
+                bail!(span,
+                    "duplicate tag id";
+                    hint: "this is a bug in typst, please report it"
+                );
+            }
+            KrillaError::UnknownTagId(_, loc) => {
+                let span = to_span(loc);
+                bail!(span,
+                    "unknown tag id";
+                    hint: "this is a bug in typst, please report it"
+                );
             }
         },
     }
@@ -540,12 +569,12 @@ fn convert_error(
         }
         // The below errors cannot occur yet, only once Typst supports full PDF/A
         // and PDF/UA. But let's still add a message just to be on the safe side.
-        ValidationError::MissingAnnotationAltText => error!(
+        ValidationError::MissingAnnotationAltText(_) => error!(
             Span::detached(),
             "{prefix} missing annotation alt text";
             hint: "please report this as a bug"
         ),
-        ValidationError::MissingAltText => error!(
+        ValidationError::MissingAltText(_) => error!(
             Span::detached(),
             "{prefix} missing alt text";
             hint: "make sure your images and equations have alt text"
@@ -581,17 +610,13 @@ fn convert_error(
             "{prefix} missing document date";
             hint: "set the date of the document"
         ),
-        ValidationError::DuplicateTagId(_, loc) => error!(
-            to_span(*loc),
-            "{prefix} duplicate tag id";
-            hint: "please report this as a bug"
-        ),
-        ValidationError::UnknownTagId(_, loc) => error!(
-            to_span(*loc),
-            "{prefix} unknown tag id";
-            hint: "please report this as a bug"
-        ),
-        ValidationError::EmbeddedPDF(loc) => error!(to_span(*loc), "TODO"),
+        ValidationError::EmbeddedPDF(loc) => {
+            error!(
+                to_span(*loc),
+                "embedding PDFs is currently not supported in this export mode";
+                hint: "try converting the PDF to SVG before embedding it"
+            )
+        }
     }
 }
 

crates/typst-pdf/src/image.rs

@@ -10,7 +10,7 @@ use typst_library::diag::{bail, SourceResult};
 use typst_library::foundations::Smart;
 use typst_library::layout::{Abs, Angle, Ratio, Size, Transform};
 use typst_library::visualize::{
-    ExchangeFormat, Image, ImageKind, ImageScaling, RasterFormat, RasterImage,
+    ExchangeFormat, Image, ImageKind, ImageScaling, PdfImage, RasterFormat, RasterImage,
 };
 use typst_syntax::Span;
 
@@ -62,13 +62,7 @@ pub(crate) fn handle_image(
             );
         }
         ImageKind::Pdf(pdf) => {
-            let pdf_data: Arc<dyn AsRef<[u8]> + Send + Sync> =
-                Arc::new(pdf.data().clone());
-            surface.draw_pdf_page(
-                &PdfDocument::new(pdf_data.into()).unwrap(),
-                size.to_krilla(),
-                pdf.page_index(),
-            )
+            surface.draw_pdf_page(&convert_pdf(pdf), size.to_krilla(), pdf.page_index())
         }
     }
 
@@ -95,9 +89,9 @@ struct Repr {
 
 /// A wrapper around `RasterImage` so that we can implement `CustomImage`.
 #[derive(Clone)]
-struct PdfImage(Arc<Repr>);
+struct PdfRasterImage(Arc<Repr>);
 
-impl PdfImage {
+impl PdfRasterImage {
     pub fn new(raster: RasterImage) -> Self {
         Self(Arc::new(Repr {
             raster,
@@ -107,7 +101,7 @@ impl PdfImage {
     }
 }
 
-impl Hash for PdfImage {
+impl Hash for PdfRasterImage {
     fn hash<H: Hasher>(&self, state: &mut H) {
         // `alpha_channel` and `actual_dynamic` are generated from the underlying `RasterImage`,
         // so this is enough. Since `raster` is prehashed, this is also very cheap.
@@ -115,7 +109,7 @@ impl Hash for PdfImage {
     }
 }
 
-impl CustomImage for PdfImage {
+impl CustomImage for PdfRasterImage {
     fn color_channel(&self) -> &[u8] {
         self.0
             .actual_dynamic
@@ -206,10 +200,15 @@ fn convert_raster(
             interpolate,
         )
     } else {
-        krilla::image::Image::from_custom(PdfImage::new(raster), interpolate)
+        krilla::image::Image::from_custom(PdfRasterImage::new(raster), interpolate)
     }
 }
 
+#[comemo::memoize]
+fn convert_pdf(pdf: &PdfImage) -> PdfDocument {
+    PdfDocument::new(pdf.pdf().clone())
+}
+
 fn exif_transform(image: &RasterImage, size: Size) -> (Transform, Size) {
     let base = |hp: bool, vp: bool, mut base_ts: Transform, size: Size| {
         if hp {

crates/typst-pdf/src/link.rs

@@ -49,7 +49,6 @@ pub(crate) fn handle_link(
             fc.push_annotation(
                 LinkAnnotation::new(
                     rect,
-                    None,
                     Target::Action(Action::Link(LinkAction::new(u.to_string()))),
                 )
                 .into(),
@@ -64,7 +63,6 @@ pub(crate) fn handle_link(
                 fc.push_annotation(
                     LinkAnnotation::new(
                         rect,
-                        None,
                         Target::Destination(krilla::destination::Destination::Named(
                             nd.clone(),
                         )),
@@ -83,7 +81,6 @@ pub(crate) fn handle_link(
         fc.push_annotation(
             LinkAnnotation::new(
                 rect,
-                None,
                 Target::Destination(krilla::destination::Destination::Xyz(
                     XyzDestination::new(index, pos.point.to_krilla()),
                 )),

crates/typst-render/Cargo.toml

@@ -18,7 +18,7 @@ typst-macros = { workspace = true }
 typst-timing = { workspace = true }
 bytemuck = { workspace = true }
 comemo = { workspace = true }
-hayro-render = { workspace = true }
+hayro = { workspace = true }
 image = { workspace = true }
 pixglyph = { workspace = true }
 resvg = { workspace = true }

crates/typst-render/src/image.rs

@@ -1,12 +1,12 @@
-use std::sync::Arc;
-use hayro_render::{InterpreterSettings, RenderSettings};
+use hayro::{FontData, FontQuery, InterpreterSettings, RenderSettings, StandardFont};
 use image::imageops::FilterType;
 use image::{GenericImageView, Rgba};
+use std::sync::Arc;
 use tiny_skia as sk;
 use tiny_skia::IntSize;
 use typst_library::foundations::Smart;
 use typst_library::layout::Size;
-use typst_library::visualize::{Image, ImageKind, ImageScaling};
+use typst_library::visualize::{Image, ImageKind, ImageScaling, PdfImage};
 
 use crate::{AbsExt, State};
 
@@ -86,7 +86,7 @@ fn build_texture(image: &Image, w: u32, h: u32) -> Option<Arc<sk::Pixmap>> {
                 let Rgba([r, g, b, a]) = src;
                 *dest = sk::ColorU8::from_rgba(r, g, b, a).premultiply();
             }
-            
+
             texture
         }
         ImageKind::Svg(svg) => {
@@ -97,26 +97,61 @@ fn build_texture(image: &Image, w: u32, h: u32) -> Option<Arc<sk::Pixmap>> {
                 h as f32 / tree.size().height(),
             );
             resvg::render(tree, ts, &mut texture.as_mut());
-            
+
             texture
         }
-        ImageKind::Pdf(pdf) => {
-            let hayro_pix = pdf.with_page(|page| {
-                // TODO: Configure so that PDF standard fonts can be resolved
-                let interpreter_settings = InterpreterSettings::default();
-                let render_settings = RenderSettings {
-                    x_scale: w as f32 / pdf.width(),
-                    y_scale: h as f32 / pdf.height(),
-                    width: Some(w as u16),
-                    height: Some(h as u16)
-                };
-                hayro_render::render(page, &interpreter_settings, &render_settings)
-                
-            });
-
-            sk::Pixmap::from_vec(hayro_pix.data_as_u8_slice().to_vec(), IntSize::from_wh(w, h)?)?
-        },
+        ImageKind::Pdf(pdf) => build_pdf_texture(pdf, w, h)?,
     };
-    
+
     Some(Arc::new(texture))
 }
+
+// Keep this in sync with `typst-svg`!
+fn build_pdf_texture(pdf: &PdfImage, w: u32, h: u32) -> Option<sk::Pixmap> {
+    let sf = pdf.standard_fonts().clone();
+
+    let select_standard_font = move |font: StandardFont| -> Option<FontData> {
+        let bytes = match font {
+            StandardFont::Helvetica => sf.helvetica.normal.clone(),
+            StandardFont::HelveticaBold => sf.helvetica.bold.clone(),
+            StandardFont::HelveticaOblique => sf.helvetica.italic.clone(),
+            StandardFont::HelveticaBoldOblique => sf.helvetica.bold_italic.clone(),
+            StandardFont::Courier => sf.courier.normal.clone(),
+            StandardFont::CourierBold => sf.courier.bold.clone(),
+            StandardFont::CourierOblique => sf.courier.italic.clone(),
+            StandardFont::CourierBoldOblique => sf.courier.bold_italic.clone(),
+            StandardFont::TimesRoman => sf.times.normal.clone(),
+            StandardFont::TimesBold => sf.times.bold.clone(),
+            StandardFont::TimesItalic => sf.times.italic.clone(),
+            StandardFont::TimesBoldItalic => sf.times.bold_italic.clone(),
+            StandardFont::ZapfDingBats => sf.zapf_dingbats.clone(),
+            StandardFont::Symbol => sf.symbol.clone(),
+        };
+
+        bytes.map(|d| {
+            let font_data: Arc<dyn AsRef<[u8]> + Send + Sync> = Arc::new(d.clone());
+
+            font_data
+        })
+    };
+
+    let interpreter_settings = InterpreterSettings {
+        font_resolver: Arc::new(move |query| match query {
+            FontQuery::Standard(s) => select_standard_font(*s),
+            FontQuery::Fallback(f) => select_standard_font(f.pick_standard_font()),
+        }),
+        warning_sink: Arc::new(|_| {}),
+    };
+    let page = pdf.page();
+
+    let render_settings = RenderSettings {
+        x_scale: w as f32 / pdf.width(),
+        y_scale: h as f32 / pdf.height(),
+        width: Some(w as u16),
+        height: Some(h as u16),
+    };
+
+    let hayro_pix = hayro::render(page, &interpreter_settings, &render_settings);
+
+    sk::Pixmap::from_vec(hayro_pix.take_u8(), IntSize::from_wh(w, h)?)
+}

crates/typst-svg/Cargo.toml

@@ -21,6 +21,7 @@ base64 = { workspace = true }
 comemo = { workspace = true }
 ecow = { workspace = true }
 flate2 = { workspace = true }
+hayro = { workspace = true }
 image = { workspace = true }
 ttf-parser = { workspace = true }
 xmlparser = { workspace = true }

crates/typst-svg/src/image.rs

@@ -1,10 +1,14 @@
+use std::borrow::Cow;
+use std::sync::Arc;
+
 use base64::Engine;
 use ecow::{eco_format, EcoString};
+use hayro::{FontData, FontQuery, InterpreterSettings, RenderSettings, StandardFont};
 use image::{codecs::png::PngEncoder, ImageEncoder};
 use typst_library::foundations::Smart;
 use typst_library::layout::{Abs, Axes};
 use typst_library::visualize::{
-    ExchangeFormat, Image, ImageKind, ImageScaling, RasterFormat,
+    ExchangeFormat, Image, ImageKind, ImageScaling, PdfImage, RasterFormat,
 };
 
 use crate::SVGRenderer;
@@ -44,7 +48,7 @@ pub fn convert_image_scaling(scaling: Smart<ImageScaling>) -> Option<&'static st
 #[comemo::memoize]
 pub fn convert_image_to_base64_url(image: &Image) -> EcoString {
     let mut buf;
-    let (format, data): (&str, &[u8]) = match image.kind() {
+    let (format, data): (&str, Cow<[u8]>) = match image.kind() {
         ImageKind::Raster(raster) => match raster.format() {
             RasterFormat::Exchange(format) => (
                 match format {
@@ -53,7 +57,7 @@ pub fn convert_image_to_base64_url(image: &Image) -> EcoString {
                     ExchangeFormat::Gif => "gif",
                     ExchangeFormat::Webp => "webp",
                 },
-                raster.data(),
+                Cow::Borrowed(raster.data()),
             ),
             RasterFormat::Pixel(_) => ("png", {
                 buf = vec![];
@@ -62,11 +66,29 @@ pub fn convert_image_to_base64_url(image: &Image) -> EcoString {
                     encoder.set_icc_profile(icc_profile.to_vec()).ok();
                 }
                 raster.dynamic().write_with_encoder(encoder).unwrap();
-                buf.as_slice()
+                Cow::Borrowed(buf.as_slice())
             }),
         },
-        ImageKind::Svg(svg) => ("svg+xml", svg.data()),
-        ImageKind::Pdf(_) => todo!(),
+        ImageKind::Svg(svg) => ("svg+xml", Cow::Borrowed(svg.data())),
+        ImageKind::Pdf(pdf) => {
+            // To make sure the image isn't pixelated, we always scale up so the lowest
+            // dimension has at least 1000 pixels. However, we only scale up as much so that the
+            // largest dimension doesn't exceed 3000 pixels.
+            const MIN_RES: f32 = 1000.0;
+            const MAX_RES: f32 = 3000.0;
+
+            let base_width = pdf.width();
+            let w_scale = (MIN_RES / base_width).max(MAX_RES / base_width);
+            let base_height = pdf.height();
+            let h_scale = (MIN_RES / base_height).min(MAX_RES / base_height);
+
+            let total_scale = w_scale.min(h_scale);
+
+            let width = (base_width * total_scale).ceil() as u32;
+            let height = (base_height * total_scale).ceil() as u32;
+
+            ("png", Cow::Owned(pdf_to_png(pdf, width, height)))
+        }
     };
 
     let mut url = eco_format!("data:image/{format};base64,");
@@ -74,3 +96,54 @@ pub fn convert_image_to_base64_url(image: &Image) -> EcoString {
     url.push_str(&data);
     url
 }
+
+// Keep this in sync with `typst-png`!
+#[comemo::memoize]
+fn pdf_to_png(pdf: &PdfImage, w: u32, h: u32) -> Vec<u8> {
+    let sf = pdf.standard_fonts().clone();
+
+    let select_standard_font = move |font: StandardFont| -> Option<FontData> {
+        let bytes = match font {
+            StandardFont::Helvetica => sf.helvetica.normal.clone(),
+            StandardFont::HelveticaBold => sf.helvetica.bold.clone(),
+            StandardFont::HelveticaOblique => sf.helvetica.italic.clone(),
+            StandardFont::HelveticaBoldOblique => sf.helvetica.bold_italic.clone(),
+            StandardFont::Courier => sf.courier.normal.clone(),
+            StandardFont::CourierBold => sf.courier.bold.clone(),
+            StandardFont::CourierOblique => sf.courier.italic.clone(),
+            StandardFont::CourierBoldOblique => sf.courier.bold_italic.clone(),
+            StandardFont::TimesRoman => sf.times.normal.clone(),
+            StandardFont::TimesBold => sf.times.bold.clone(),
+            StandardFont::TimesItalic => sf.times.italic.clone(),
+            StandardFont::TimesBoldItalic => sf.times.bold_italic.clone(),
+            StandardFont::ZapfDingBats => sf.zapf_dingbats.clone(),
+            StandardFont::Symbol => sf.symbol.clone(),
+        };
+
+        bytes.map(|d| {
+            let font_data: Arc<dyn AsRef<[u8]> + Send + Sync> = Arc::new(d.clone());
+
+            font_data
+        })
+    };
+
+    let interpreter_settings = InterpreterSettings {
+        font_resolver: Arc::new(move |query| match query {
+            FontQuery::Standard(s) => select_standard_font(*s),
+            FontQuery::Fallback(f) => select_standard_font(f.pick_standard_font()),
+        }),
+        warning_sink: Arc::new(|_| {}),
+    };
+    let page = pdf.page();
+
+    let render_settings = RenderSettings {
+        x_scale: w as f32 / pdf.width(),
+        y_scale: h as f32 / pdf.height(),
+        width: Some(w as u16),
+        height: Some(h as u16),
+    };
+
+    let hayro_pix = hayro::render(page, &interpreter_settings, &render_settings);
+
+    hayro_pix.take_png()
+}