Vista previa de PDF en línea

zogstrip · 15 Julio, 2026 08:47

@tpetrov esto ya está corregido

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.

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

Vista previa de PDF en línea

Temas relacionados