search

CNRS-LACITO / eastlingplayer

This is the official repository of the Eastling player. It is part of the Eastling suite: Easy Annotation and Synchronization Tool for linguists.
GNU General Public License v3.0
2 stars 0 forks source link

pas d'affichage des boutons DOI dans Firefox #54

Closed alexis-michaud closed 3 years ago

alexis-michaud commented 3 years ago

Le bouton est bien actif. Quand on clique au bon endroit, le DOI s'affiche. Mais l'image (icône DOI) ne s'affiche pas.

image

alexis-michaud commented 3 years ago

Même souci relevé par @sguillaume : le logo des DOIs n'apparait pas dans firefox devant les phrases.

firefox_pas_de_doi

alexis-michaud commented 3 years ago

(Ce serait pas mal d'être compatible avec Firefox)

EdouardSombie commented 3 years ago

C'est parce que Firefox n'accepte pas de pseudo element sur la balise img... Du coup, il faut faire une modif sur le player : @Matthew : il faudrait remettre une source sur les images qui ont la class "doi" comme ci-dessous :

doi

Tu pourras me faire signe quand c'est en ligne ? Il faudra que je mette à jour le css. A tout à l'heure

EdouardSombie commented 3 years ago

Github a interpréter le html ... Revoici la balise img: <img class="doi" src="/dist/images/DOI_logo.svg" alt="doi">

m8nli9ht commented 3 years ago

traité et livré, à tester

m8nli9ht commented 3 years ago

Etrangement, l'image ne s'affiche toujours pas mais on la voit bien dans l'inspecteur de code de Firefox : image

m8nli9ht commented 3 years ago

En définissant une largeur et une hauteur ça s'affiche : image

m8nli9ht commented 3 years ago

@EdouardSombie combien faut-il de pixels pour width et height du coup ?

EdouardSombie commented 3 years ago

@m8nli9ht : pas besoin de spécifier de width ou de height

EdouardSombie commented 3 years ago

J'avais mis une width et une height de 0 pour remplacer l'image par un pseudo élément. Je vais modifier le css pour que l'image s'affiche bien.

EdouardSombie commented 3 years ago

Réglé :)