Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout de la qualité (tooltip et composant texte) #468

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

paulop33
Copy link
Contributor

@paulop33 paulop33 commented Nov 10, 2024

Bonjour,

Dans le but de faire avancer la gestion des forks, j'ai commencé à intégrer la notion de qualité en m'inspirant de ce que Montpellier a fait.
Le but n'est évidemment pas de forcer tout le monde à afficher/gérer une notion de qualité par tronçon.

Au contraire, cette fonctionnalité est désactivée par défaut. Elle n'a donc aucun impact sur le site lyonnais.
Dans config.json, il faudra ajouter "qualityDisplay": true pour activer la fonctionnalité.
Enfin, aucune information n'est affichée si aucun élément n'est défini (y compris si la fonctionnalité est activée)

J'ai ajouté un composant QualityText qui permet d'ajouter un texte simple :

### Pont Clémenceau à Boulevard des Belges
:qualityText{type="perfect"} *via Tunnel mode doux et avenue Duquesne*

image

Pour la carte interactive, ajouter la property quality permet son affichage dans le LineTooltip

    {
      "type": "Feature",
      "properties": {
        "id": "PontClemenceau-commun",
        "quality": "perfect"
      },
    },

image
En son absence, la ligne ne s'affiche pas dans la boite :
image

6 valeurs sont possibles :
dangerous: 'Dangereuse',
bad: 'Non satisfaisante',
fair: 'Globalement ',
good: 'Satisfaisante',
perfect: 'Parfaite',
unknown: 'Inconnue'

@paulop33 paulop33 force-pushed the feat/quality-text-and-tooltip branch from d0a9ae2 to 037f0e3 Compare November 10, 2024 12:15
@paulop33 paulop33 changed the title [Gestion fork] Ajout de la qualité (tooltip et composant Texte) [Gestion fork] Ajout de la qualité (tooltip et composant texte) Nov 10, 2024
@paulop33
Copy link
Contributor Author

paulop33 commented Nov 11, 2024

Suite aux retours de @ThibautChd
imageimageimage
J'ai dû réduire le texte car "Non satisfaisant" ne rentrait pas.
Il faudra peut être utiliser un autre svg (en l'état, c'est celui de tailwind).

Plus le composant texte (à utiliser ou pas) :
image
image

C'est prêt à être mergé.
L'option reste à activer dans la config. En l'état, sans activation ni donnée, rien ne s'affiche sur le site.

Reste à développer le reste ^^

#406

@paulop33 paulop33 marked this pull request as ready for review November 11, 2024 20:45
@paulop33 paulop33 force-pushed the feat/quality-text-and-tooltip branch from 7bb1fa4 to 6c5e82a Compare November 11, 2024 21:29
@paulop33
Copy link
Contributor Author

paulop33 commented Nov 11, 2024

Petit bonus du soir, je viens de créer et ajouter le bloc de stats qualité sur la page de texte :
image

#406

@paulop33 paulop33 changed the title [Gestion fork] Ajout de la qualité (tooltip et composant texte) Ajout de la qualité (tooltip et composant texte) Nov 11, 2024
@paulop33
Copy link
Contributor Author

Ajout de la qualité générale sur la page d'accueil
Son affichage est conditionné par qualityDisplay et qualityDisplayOnHomePage dans config.json

image

QualityText component

possibilité de déclarer la qualité de l'aménagement et affichage sur tooltip

LineTooltip : si la donnée qualité n'est pas définie, le LineTooltip n'affiche pas de ligne supplémentaire

qualityText : notion de Inconnue manquante fix

factorisation des label de qualityNames

qualité : réduction du nombre d'état (satisfaisant et non satisfaisant) + adapatation du LineTooltip

composant StatsQuality créé et ajouté dans le bloc aperçu de la page des lignes

ajout du composant qualité sur la page d'accueil
@paulop33 paulop33 force-pushed the feat/quality-text-and-tooltip branch from b92672f to c943329 Compare November 27, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant