With pdf.js and disableFontFace: true, fonts are too large #3058

shlyakpavel · 2024-12-27T16:47:53Z

Summary

When I open pdf.js in Ladybird with disableFontFace: true, fonts are too large

Operating system

macOS

Steps to reproduce

Open Ladybird
Navigate to a webpage like the one in the repro
Observe

Expected behavior

It renders fine

Actual behavior

Fonts are too large

URL for a reduced test case

N/A

HTML/SVG/etc. source for a reduced test case

<!DOCTYPE html>
<html>
<head>
    <title>PDF.js Test</title>
    <script type="module">
        import * as pdfjsLib from './pdf.mjs';

        pdfjsLib.GlobalWorkerOptions.workerSrc = new URL('pdf.worker.mjs', import.meta.url).href;

        const url = 'what-has-government-done-to-our-money.pdf';

        const canvas = document.getElementById('pdf-canvas');
        const ctx = canvas.getContext('2d');

		pdfjsLib.getDocument({
			url,
			ignoreErrors: true, // Prevent the rendering process from stopping
			disableFontFace: true, // Use default fonts if custom fonts are missing
		}).promise.then(function (pdf) {
			pdf.getPage(2).then(function (page) {
				const viewport = page.getViewport({ scale: 1.5 });
				canvas.width = viewport.width;
				canvas.height = viewport.height;

				const renderContext = {
					canvasContext: ctx,
					viewport: viewport,
				};

				page.render(renderContext);
			});
		});
    </script>
</head>
<body>
    <canvas id="pdf-canvas"></canvas>
</body>
</html>

Log output and (if possible) backtrace

N/A

Screenshots or screen recordings

Build flags or config settings

No response

Contribute a patch?

I’ll contribute a patch for this myself.

AtkinsSJ added bug Something isn't working has repro We have a way to reproduce this bug. needs reduction Issue needs someone to make a minimal test case that demonstrates the problem. labels Jan 9, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

With pdf.js and disableFontFace: true, fonts are too large #3058

With pdf.js and disableFontFace: true, fonts are too large #3058

shlyakpavel commented Dec 27, 2024 •

edited

Loading

With pdf.js and disableFontFace: true, fonts are too large #3058

With pdf.js and disableFontFace: true, fonts are too large #3058

Comments

shlyakpavel commented Dec 27, 2024 • edited Loading

Summary

Operating system

Steps to reproduce

Expected behavior

Actual behavior

URL for a reduced test case

HTML/SVG/etc. source for a reduced test case

Log output and (if possible) backtrace

Screenshots or screen recordings

Build flags or config settings

Contribute a patch?

shlyakpavel commented Dec 27, 2024 •

edited

Loading