← Retour aux issues

documentation et refacto du style france avec correction de bugs

publié le , mis à jour

⚙️ Ceci est une proposition de changement de code.

Rendez-vous sur sa page Coderberg pour l'osculter.

Avatar Codeberg de etienneJretienneJr

du boulot dans france.ts :

  • documentation (surtout pour les layers de POIS)
  • correction du bug de l'icone des gares qui ne s'affichait plus
  • stockage dans des constantes de différentes expressions utilisées plusieurs fois
    • l'expression pour la visibilité des gares
    • l'expression pour la visibilité des POIs
    • l'expression pour calculer le nom de l'icone
  • suppression du layer des hôpitaux, en le gérant dans le layer Other POI
  • fix #1000 : parkings vélos un peu plus visibles
  • fix #1014 : afficher les POIs plus tôt (sans distinction mobile ou desktop pour l'instant)
  • fix #992 et fix #967 : correction des tunnels, dont meilleure visibilité du tunnel sous la Manche
  • fix #1008 : décalage des landuse vers le fond pour faire apparaitre les bassins
  1. Avatar Codeberg de etienneJretienneJr

    @laem c'est bon pour moi pour cette PR. Je ne fusionne pas moi même à cause de https://codeberg.org/cartes/web/pulls/1024/commits/9edc01ebcbe38f75c3416c92e6582de5ec0d060c qui a modifié deno.lock. Je ne sais pas pourquoi ça ne s'est pas fait chez toi quand tu as ajouté osm-api ?

  2. Avatar Codeberg de laemlaem

    Je ne sais pas pourquoi ça ne s'est pas fait chez toi quand tu as ajouté osm-api

    En fait pour l'application web c'est Bun qu'on utilise.

    Deno n'est là que pour exécuter certains scripts. Sa librairie standard est bien meilleure, globalement c'est un meilleur projet je trouve.

    Là où Bun n'est qu'un node plus rapide qui cherche avant tout la compatibilité.

    On utilise seulement Deno pour le script update-icons, donc osm-api n'y était pas utilisé, si ?

  3. Avatar Codeberg de laemlaem

    Géniale cette PR, t'as corrigé plein de trucs, bravo 👏.


✏️ Participer à la discussion