Embora isso seja muito funcional, adiciona algumas etapas desnecessárias para o caso de uso mais comum: “Eu quero ler isso agora” — especialmente em dispositivos móveis.
A necessidade de baixar/salvar algo é muito secundária. Às vezes é necessária, mas geralmente o usuário apenas quer ler e seguir em frente.
O TC em Inline PDF Previews aborda isso ao servir um PDF embutido no desktop — e faz isso com excelência. No entanto, ele ignora essas situações comuns:
PDFs em dispositivos móveis
Múltiplos PDFs (devido ao excesso visual)
Visualização de PDFs em uma aba completa do navegador
Vale notar que existem outros caminhos para baixar um PDF. Eles podem ser baixados diretamente de um link (por meio de clique com o botão direito / menu de contexto) ou a partir do visualizador de PDF baseado em navegador que este TC fornece.
Sugestão: abrir PDFs no navegador por padrão
Essa pequena alteração resolve bem ambas as situações e simplesmente se integra ao comportamento padrão para outros links no Discourse. Assim, com a mudança proposta:
Links de PDF em dispositivos móveis normalmente abrem na mesma janela do navegador
permite visualização com um único toque
Links de PDF no desktop (ou seja, acima do PDF embutido) abrem em uma nova aba
permite visualização em uma aba completa com um único clique
Quando o comportamento embutido é deliberadamente suprimido (adicionando um espaço no texto do link)
Aqui em meta.discourse.org, meu link acima abre na mesma aba desta página (exatamente como você descreve). Isso não muda em safe-mode.
No entanto, em três dos meus sites atualizados recentemente (com o TC desativado), ele tenta um download. Isso também acontece com o safe-mode. Não sei por que isso ocorre. Não consigo ver nenhuma configuração de site que governe isso.
Qualquer que seja o motivo, meu fork corrige isso bem para os meus sites!
Nós temos um trabalho em andamento. Esperamos conseguir implementá-lo nas próximas semanas. É um pouco complicado porque há muitas situações diferentes para verificar (uploads locais, s3, compatível com s3, com cdn, sem cdn, etc.).
Acabei de verificar um PDF que carreguei em um tópico hoje em um site recentemente atualizado e clicar no link do PDF abre a URL do PDF no CDN no meu navegador, só para constar.
Agora temos lógica centralizada para determinar quais arquivos devem ser mostrados ‘inline’. Isso significa que PDFs são consistentemente mostrados inline, e alguns tipos de arquivo menos seguros são consistentemente servidos como downloads. Essas mudanças devem funcionar em todos os tipos de armazenamento de upload (local e S3, com ou sem CDNs).
David, acabei de testar isto - e embora isto esteja definitivamente corrigido no desktop, não parece ser o caso para o celular.
Quando clico em um link de PDF no celular, recebo um download (o que eu não quero). Eu quero que ele abra diretamente no navegador (ou seja, o mesmo que no desktop), independentemente de o TC estar instalado ou não.
Nenhuma alteração desde a correção de segurança vinculada acima. Poderia ter sido cache, especialmente se você fez a atualização pela interface do usuário na primeira vez. Se você fez uma reconstrução completa agora, isso terá redefinido o cache do NGINX. Que bom saber que está funcionando agora!
