128 missing meetup ressources

ToulouseDataViz / website

L'espace de dévelopment du site web toulouse-dataviz.fr

2 stars 2 forks source link

128 missing meetup ressources #132

Closed emaulandi closed 2 years ago

emaulandi commented 2 years ago

voir #128

ressources reveal ajoutées
ajout de dossiers dans /static
d'autres modifications de liens ont été effectué sur notion dans la table meetup full
des présentation ont été ajoutées dans le repo principal / presentations

vvivanloc commented 2 years ago

It won't compile :smile:

emaulandi commented 2 years ago

Je vais corriger le warning (un http qui devrait être un htpps dans les reveal)

@vvivanloc il y a 2 images restantes dans /static, j'ai l'impression que c'est des images de la revue de presse, qui sont en fait dans src/content/pic/pressreview. Est-ce qu'on a toujours besoin de les avoir dans /static ?

vvivanloc commented 2 years ago

Oui, il me semble qu'ils sont nécessaires : cf. fichier src/data/press_reviews.csv.

Les miniatures sont dans le dossier website/src/content/pics/other (j'imagine que tu as dû renommer ce dossier pressreview dans ta branche ?)
Les images en taille réelle sont dans le dossier /static

emaulandi commented 2 years ago

Oui c'est src/content/pics/other désolé, dans ma tête c'était press review (le nom du csv), mais je n'ai pas renommé.

A priori il n'y aurait pas besoin d'avoir ces images en taille réelle ? Dans la section de la revue de presse, au survol les images s'agrandissent comme les autre images de la gallerie. Est-ce qu'on ne mettrait pas autre chose en lien (ex: le conseil departemental) au clic sur "Plus de détails", ou alors rien du tout ? Je trouve que ça n'apporte pas une plue value énorme d'ouvrir l'image seule dans la fenetre en plus de l'agrandissement, mais c'est juste mon avis et du détail :)

emaulandi commented 2 years ago

@vvivanloc c'est bon pour toi ?

vvivanloc commented 2 years ago

C'est bon ! Merci pour le boulot !

vvivanloc commented 2 years ago

@emaulandi Je te laisse supprimer ta branche.