Changement graphique #10
Conversation
There was a problem hiding this comment.
Je pense que cela pourrait être pas-mal de mettre cela en texte dans la légende.
| </p> | ||
| </div> | ||
|
|
||
| <div id="infos-pts"> | ||
| <div> | ||
| <h2 id="pt-obs"></h3> | ||
| <h3>Qu'observons nous ?</h3> | ||
| <p id="what-do-we-see"></p> | ||
| <p style="text-align: justify;", id="what-do-we-see"></p> |
There was a problem hiding this comment.
Posibilité de simplifier la redondance de définition des styles en ajoutant dans le CSS :
#infos-pts p {
text-align: justify
}| @@ -140,13 +143,21 @@ map.on('load', () => { | |||
| map.addLayer({ | |||
| "id": "pts", | |||
| "type": "circle", | |||
| "width": "3px", | |||
There was a problem hiding this comment.
Préférer une unité adaptative, genre le %, le pixel ne change pas en fonction de la taille de l'écran
|
|
||
| // Écouteurs d'événements pour les cases à cocher |
There was a problem hiding this comment.
Super ! c'est très lisible.
Petit tips, tu n'es pas obligé de mettre en commentaire le code précédent, dans la mesure où Git permet de garder en mémoire ce code. (il faut éviter au maximum de garder du code commenté)
| } | ||
|
|
||
| #infos-pts p { | ||
| margin-top: 1%; | ||
| font-size: 65%; | ||
| } |
There was a problem hiding this comment.
Rapport à ce que j'ai dit plus haut, tu peux indiquer ici directement :
| } | |
| text-align: justify | |
| } |
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Même remarque sur les pixels
No description provided.