TMS - curseur de définition de l'échelle de zoom - Mineur

IGNF / cartes.gouv.fr

Le portail principal d'accès à la Géoplateforme, son entrepôt et son catalogue

GNU Affero General Public License v3.0

3 stars 0 forks source link

TMS - curseur de définition de l'échelle de zoom - Mineur #362

Open IGNF-Xavier opened 5 months ago

IGNF-Xavier commented 5 months ago

Le besoin que vous rencontrez

Vraiment très mineur. Faire en sorte que le curseur utilise le nouveau composant du DSFR : https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/curseur-range

La solution que vous aimeriez

No response

Les alternatives que vous avez considéré

No response

Autres éléments de contexte

No response

ocruze commented 4 months ago

Une comparaison du curseur MUI (première ligne, celui qui est implémenté aujourd'hui) et DSFR (deuxième ligne) :

Quelques limitations du curseur DSFR (https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/curseur-range) :

on ne voit pas toutes les étapes numérotées (j'ai demandé sur slack si c'est prévu dans le futur, on verra ce qu'ils disent)
il n'y a pas les crans quand on est en mode double
on ne peut pas cliquer n'importe où sur la piste horizontale en mode double

Dans tous les cas, on attend un peu avant d'utiliser le curseur DSFR parce qu'il y a un bug qui fait planter la page de temps en temps.

IGNF-Xavier commented 4 months ago

Pas de soucis, je l'ai mis en très mineur et en MVP-futur. C'est loin d'être une priorité. D'autres tickets sont plus importants dont ceux des compteurs qui empêchent un certain nb de tests.

slafayIGN commented 4 months ago

Les 3 changements listés ne sont pas vraiment problématiques. C'est un choix du DSFR qui allège que composant. Hormis cette histoire de bug qui semble un peu aléatoire, je ne vois pas vraiment de contre-indication à utiliser ce nouveau composant.

slafayIGN commented 4 months ago

@pprev94 a repéré un bug dans le composant quand les 2 curseurs sont au même endroit : https://github.com/codegouvfr/react-dsfr/issues/259. Une seule des 2 valeurs est modifiée quand un curseur "pousse" l'autre et ça peut mener à l'enregistrement de valeurs incohérentes avecce que visualise l'utilisateur et à des zoom min > zoom max.

pprev94 commented 3 months ago

La correction a été faites dans dsfr

slafayIGN commented 3 months ago

TODO : mettre à jour la dépendance à react-dsfr pour avoir le DSFR >= 1.12.0

slafayIGN commented 1 week ago

La dépendance au DSFR étant à jour pour inclure la correction du bug (1.12.1), on peut reprendre le changement du composant par celui du DSFR.