Aperçus PDF en ligne

zogstrip · Juillet 15, 2026, 8:47

@tpetrov c’est maintenant corrigé

github.com/discourse/discourse-pdf-previews

DEV: Move mobile-view check out of initialization (#60)

main ← fix-static-viewport-deprecation

merged 08:46AM - 15 Jul 26 UTC

ZogStriP

+5 -4

### What & why The component read `site.mobileView` inside its initializer, whi…ch runs while the application is still booting. Accessing the viewport that early is deprecated (`discourse.static-viewport-initialization`) — the value isn't reliable before boot and can't react to viewport changes. As a result, every admin on a site running this component saw a persistent **"[Admin Notice] … contains code which needs updating"** banner. ### Fix - Move the desktop-only guard into the `decorateCookedElement` callback so it is evaluated at render time instead of during initialization. - Read it from the modern `capabilities.viewport` service rather than the legacy mobile/desktop split. Since `site.mobileView` is defined as `!capabilities.viewport.sm`, behaviour is unchanged — previews remain desktop-only. See https://meta.discourse.org/t/367810 for the viewport guidance.

Sujet		Réponses	Vues
Discourse send PDF inline Plugin	11	1646	Février 17, 2026
Make PDFs open directly (not via download) by default Feature completed , pdf-previews	15	548	Février 18, 2026
Add configuration option to serve local PDF uploads inline Feature completed , pdf-previews	2	787	Septembre 5, 2024
How to view .docs or .docx as preview or another tab in discourse Support pdf-previews	2	368	Juin 7, 2024
Regression - PDFs download initiated on click (should be open in new tab) Bug pdf-previews , fixed	1	92	Juin 10, 2026

Aperçus PDF en ligne

Sujets connexes