Should not process pages with content-type equals to "application/*+xml"

istex-archives / istex-browser-extension

Bouton ISTEX : extension web capable d'insérer dynamiquement sur la page web consultée un lien vers le fulltext d'un document si ce dernier est disponible dans la plateforme ISTEX

https://addons.istex.fr

MIT License

11 stars 1 forks source link

Should not process pages with content-type equals to "application/*+xml" #8

Closed niederlec closed 6 years ago

niederlec commented 6 years ago

This issue is almost the same as : https://github.com/istex/istex-browser-extension/commit/bf7dda036afe16b4ceb044f278d06c77455daa12#diff-989f36bb0ca710eb5fb30a41d09827a6

niederlec commented 6 years ago

Example of page with XML broken display : https://www.idref.fr/139753753.rdf

kerphi commented 6 years ago

Pour illustrer :

Sans l'extension ça donne :

Avec l'extension ça donne :

niederlec commented 6 years ago

j'allais clore le ticket mais tu me mets le doute... As-tu testé la version 1.3.1 que j'ai publiée cet après-midi ? PS : j'aime bien le choix de l'exemple ;-)

kerphi commented 6 years ago

Non non tu peux le fermer, j'ai ajouté les copies d'écrans de François pour montrer comment on aurait pu formuler le bug directement ici. Je ferai un petit mail demain pour y faire référence.