Aunque esto es muy funcional, añade unos cuantos pasos innecesarios para el caso de uso más común: “Quiero leer esto ahora”, especialmente en dispositivos móviles.
La necesidad de descargar o guardar algo es muy secundaria. A veces es necesario, pero normalmente el usuario simplemente quiere leerlo y seguir adelante.
El TC Inline PDF Previews aborda esto al mostrar un PDF incrustado en escritorio, y lo hace con gran efectividad. Sin embargo, omite estas situaciones comunes:
PDFs en dispositivos móviles
Múltiples PDFs (debido al desorden visual)
Visualización de PDFs en una pestaña completa del navegador
Cabe destacar que existen otras formas de descargar un PDF. Pueden descargarse directamente desde un enlace (mediante clic derecho / menú contextual) o desde el visor de PDF basado en navegador que proporciona este TC.
Sugerencia: abrir PDFs en el navegador por defecto
Este pequeño ajuste resuelve ambas situaciones de forma elegante y simplemente se alinea con el comportamiento predeterminado de otros enlaces en Discourse. Así, con el cambio propuesto:
Los enlaces a PDF en móviles suelen abrirse en la misma ventana del navegador
permite la visualización con un solo toque
Los enlaces a PDF en escritorio (es decir, por encima del PDF incrustado) se abren en una nueva pestaña
permite la visualización completa en una sola pestaña con un clic
Cuando el comportamiento incrustado se suprime deliberadamente (añadiendo un espacio en el texto del enlace)
Aquí en meta.discourse.org, mi enlace de arriba se abre en la misma pestaña que esta página (tal como describes). Esto no cambia en el modo seguro (safe-mode).
Sin embargo, en tres de mis sitios actualizados recientemente (con el TC deshabilitado) intenta una descarga. Este también es el caso con el modo seguro. No estoy seguro de por qué es así. No puedo ver ninguna configuración del sitio que gobierne esto en absoluto.
Cualquiera que sea la razón, ¡mi bifurcación lo soluciona muy bien para mis sitios!
Tenemos un trabajo en curso. Esperamos poder implementarlo en las próximas dos semanas. Es un poco complicado porque hay muchas situaciones diferentes que verificar (cargas locales, s3, compatible con s3, con cdn, sin cdn, etc.).
Acabo de revisar un PDF que subí a un tema hoy en un sitio actualizado recientemente y hacer clic en el enlace del PDF abre la URL del PDF en la CDN en mi navegador, por si sirve de algo.
Ahora tenemos lógica centralizada para determinar qué archivos deben mostrarse ‘en línea’. Esto significa que los PDF se muestran consistentemente en línea y algunos tipos de archivos menos seguros se sirven consistentemente como descargas. Estos cambios deberían funcionar en todo tipo de almacenamiento de subidas (local y S3, con o sin CDN).
David, acabo de probar esto, y aunque definitivamente parece solucionado en el escritorio, no parece ser el caso en el móvil.
Cuando hago clic en un enlace PDF en el móvil, obtengo una descarga (lo cual no quiero). Quiero que se abra directamente en el navegador (es decir, lo mismo que en el escritorio), independientemente de si el TC está instalado o no.
¿Qué pasa con los archivos mp4? En la mayoría de los sitios web, los enlaces a mp4 los reproducen de forma natural en el navegador, pero no en Discourse, lo cual es bastante molesto
No se han realizado ajustes desde la corrección de seguridad vinculada anteriormente. Podría haber sido el almacenamiento en caché, especialmente si realizaste la actualización a través de la interfaz de usuario la primera vez. Si ahora has realizado una reconstrucción completa, eso habrá restablecido la caché de NGINX. ¡Me alegra saber que ya funciona!
He abierto un nuevo tema para el problema de los mp4
