Copier l'URL de diffusion

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

Copier l'URL de diffusion #245

Closed slafayIGN closed 6 months ago

slafayIGN commented 9 months ago

Comportement à discuter, plusieurs options :

~ouvrir la page de partage des URL~ (revient à ce que fait le bouton Visualiser)
copier directement l'URL (possible uniquement s'il n'y a qu'une seule URL) : Copier l'URL du GetCapabilities filtré
~ouvrir une modale avec des input disposant chacun de leur bouton de copie~ (revient à dupliquer ce qui a été fait sur la page de visualisation)

Question à résoudre : quel système de notification pour afficher "URL copiée". Il n'y a pas de composant dédié dans le DSFR en dehors des alertes. Mais elles sont faites pour aller dans le flux de la page et non pour être "flottantes" => regarder du côté de @mui/snackbar https://mui.com/material-ui/react-snackbar/

ocruze commented 9 months ago

copier directement l'URL (possible uniquement s'il n'y a qu'une seule URL) : Copier l'URL du GetCapabilities filtré

Je propose une légère modification :

Copier l'URL ".../{z}/{x}/{y}.pbf" si flux TMS
Copier l'URL du GetCapabilities filtré si WFS ou WMS-VECTOR

AlexandreJaqua commented 8 months ago

Fonctionne comme prévu, j'ai néanmoins une réserve + une question :

[ ] Réserve : le texte de l'alerte est erroné, il faudrait soit avoir "URL copiée" soit "L'URL a été copiée", mais pas entre les deux. La première solution me semble plus clean.
[x] Question : Sur un TMS l'URL sort littéralement en ".../{z}/{x}/{y}.pbf", mais n'est-on pas supposé remplacer les variables x/y/z par quelque chose ? En l'état du coup le lien n'apporte rien.

ocruze commented 7 months ago

Le texte de l'alerte modifié à "URL copiée".

Par contre, une URL de flux TMS se termine bien avec ".../{z}/{x}/{y}.pbf". C'est le SIG (QGIS par exemple) qui va remplacer ces variables par les coordonnées de l'affichage du SIG

AlexandreJaqua commented 6 months ago

Ah oups, merci pour les explications en ce qui concerne l'URL des TMS. Par contre attention j'ai toujours une alerte "l'URL copiée" à la place de "URL copiée"

Je clos quand même le ticket puisque la fonctionnalité est OK, quitte à en refaire un plus tard avec ces petites coquilles de texte mineures.

ocruze commented 6 months ago

Ah pardon j'avais oublié de corriger le texte à un autre endroit. Maintenant c'est fait, ça sera visible à un prochain déploiement.