Bien que cela soit très fonctionnel, cela ajoute quelques étapes inutiles pour le cas d’utilisation le plus courant : « Je veux le lire maintenant » — surtout sur les appareils mobiles.
Le besoin de télécharger / enregistrer quelque chose est très secondaire. Parfois requis, mais généralement, l’utilisateur souhaite simplement le lire puis passer à autre chose.
Le TC Inline PDF Previews traite ce problème en affichant un PDF en ligne sur ordinateur — et le fait avec brio. Toutefois, il ne couvre pas ces situations courantes :
PDF sur les appareils mobiles
Plusieurs PDF (en raison du désordre visuel)
Affichage des PDF dans un onglet de navigateur complet
À noter, il existe d’autres moyens de télécharger un PDF. Ils peuvent être téléchargés directement depuis un lien (via clic droit / menu contextuel) ou depuis le visualiseur PDF basé sur le navigateur que ce TC fournit.
Proposition : ouvrir les PDF dans le navigateur par défaut
Ce petit ajustement résout agréablement ces deux situations et s’inscrit simplement dans le comportement par défaut des autres liens dans Discourse. Ainsi, avec le changement proposé :
Les liens PDF sur mobile s’ouvrent généralement dans la même fenêtre du navigateur
permet une visualisation en un seul clic
Les liens PDF sur ordinateur (c’est-à-dire au-dessus du PDF en ligne) s’ouvrent dans un nouvel onglet
permet une visualisation en un clic dans un onglet complet
Lorsque le comportement en ligne est délibérément désactivé (en ajoutant un espace dans le texte du lien)
Ici, sur meta.discourse.org, mon lien ci-dessus s’ouvre dans le même onglet que cette page (exactement comme vous le décrivez). Cela reste inchangé en mode safe-mode.
Cependant, sur trois de mes sites récemment mis à jour (avec le TC désactivé), il tente un téléchargement. C’est également le cas avec le safe-mode. Je ne sais pas pourquoi. Je ne vois aucun paramètre de site qui régisse cela.
Quelle qu’en soit la raison, mon fork le corrige bien pour mes sites !
Nous avons un travail en cours. Nous espérons pouvoir le finaliser dans les deux prochaines semaines. C’est un peu délicat car il y a beaucoup de situations différentes à vérifier (téléversements locaux, S3, compatible S3, avec CDN, sans CDN, etc.).
Je viens de vérifier un PDF que j’ai mis en ligne dans un sujet aujourd’hui sur un site récemment mis à jour et cliquer sur le lien du PDF ouvre l’URL du PDF sur le CDN dans mon navigateur, pour information.
Nous avons maintenant une logique centralisée pour déterminer quels fichiers doivent être affichés « en ligne ». Cela signifie que les PDF sont systématiquement affichés en ligne, et que certains types de fichiers moins sûrs sont systématiquement servis en téléchargement. Ces changements devraient fonctionner sur tous les types de stockage de téléchargement (local et S3, avec ou sans CDN).
David, je viens de tester ceci - et bien que cela semble définitivement corrigé sur ordinateur, ce n’est pas le cas sur mobile.
Lorsque je clique sur un lien PDF sur mobile, je reçois un téléchargement (ce que je ne veux pas). Je veux qu’il s’ouvre directement dans le navigateur (c’est-à-dire la même chose que sur ordinateur), que le TC soit installé ou non.
Qu’en est-il des fichiers mp4 ? Sur la plupart des sites web, les liens vers des mp4 les font lire naturellement dans le navigateur, mais pas dans Discourse, ce qui est plutôt ennuyeux
Aucun ajustement depuis la correction de sécurité mentionnée ci-dessus. Cela aurait pu être dû à la mise en cache, surtout si vous avez effectué la mise à niveau via l’interface utilisateur la première fois. Si vous avez maintenant effectué une reconstruction complète, cela aura réinitialisé le cache NGINX. Je suis heureux d’apprendre que cela fonctionne maintenant !
