Gestion de Leaflet en package par modules by lowzonenose · Pull Request #344 · IGNF/geoportal-extensions

lowzonenose · 2022-12-05T08:37:18Z

PLUGINS LEAFLET

Split en modules

utilisation du script webpack pour splitter en modules

$ ./node_modules/.bin/webpack --config build/webpack/webpack.config.leaflet.modules --mode=production
$ ls dist/leaflet-modules/
	Common.css    CRS.js             Isocurve.js        MousePosition.css   Route.css         WMS.js
	Common.js     ElevationPath.css  Layers.js          MousePosition.js    Route.js          WMTS.js
	Controls.css  ElevationPath.js   LayerSwitcher.css  ReverseGeocode.css  SearchEngine.css
	Controls.js   Isocurve.css       LayerSwitcher.js   ReverseGeocode.js   SearchEngine.js

taille des modules

123K  Common.css    <!-- FIXME ? -->
5,1K  Common.js     <!-- FIXME ? -->
 64K  Controls.css
570K  Controls.js   <!-- INFO tous les composants avec dependances optimisées -->
163K  CRS.js
5,3K  ElevationPath.css
171K  ElevationPath.js
 24K  Isocurve.css
119K  Isocurve.js
 27K  Layers.js      <!-- INFO tous les composants avec dependances optimisées -->
 13K  LayerSwitcher.css
 60K  LayerSwitcher.js
8,2K  MousePosition.css
243K  MousePosition.js
7,5K  ReverseGeocode.css
148K  ReverseGeocode.js
 17K  Route.css
134K  Route.js
9,5K  SearchEngine.css
 93K  SearchEngine.js
 20K  WMS.js
 20K  WMTS.js

FIXME

Common.css devrait être intégré aux widget
Common.js me semble inutile

les CSS des widgets ont besoins de Common, mais comment éviter de livrer ce fichier ?

carte des dépendances

./node_modules/.bin/webpack --config build/webpack/webpack.config.leaflet.modules --mode=none --profile --json > map-leaflet-modules.json

(nettoyer les 1ere lignes inutiles)

./node_modules/.bin/webpack-bundle-analyzer map-leaflet-modules.json

Utilisation des modules dans une page web

Il existe 2 modes pour intégrer les modules : Factory ou Classes

Factory de contrôles et de couches :

<html>
    <head>
	<link rel="stylesheet" href=".leaflet.css" />
        <script src="leaflet.js"></script>
        
        <script src="GpServices.js"></script>
        
	<script src="Layers.js"></script>
        <script src="Controls.js"></script>
        <link rel="stylesheet" href="Controls.css" />
    </head>
    <body>
        <div id="map"></div>
    	<script>
    	var createMap = function () {
	      // on cache l'image de chargement du Géoportail.
	      document.getElementById("map").style.backgroundImage = "none";

	      // Création de la map
	      var layer = L.geoportalLayer.WMTS({
		layer : "ORTHOIMAGERY.ORTHOPHOTOS"
	      });

	      var map  = L.map('map', {
		zoom : 2,
		center : L.latLng(48, 2)
	      });

	      layer.addTo(map);

	      var iso = L.geoportalControl.Isocurve();
	      map.addControl(iso);
	      var layerSwitcher = L.geoportalControl.LayerSwitcher();
	      map.addControl(layerSwitcher);
	      var mp = L.geoportalControl.MousePosition();
	      map.addControl(mp);
	      var route = L.geoportalControl.Route();
	      map.addControl(route);
	      var reverse = L.geoportalControl.ReverseGeocode();
	      map.addControl(reverse);
	      var search = L.geoportalControl.SearchEngine();
	      map.addControl(search);
	      var measureProfil = L.geoportalControl.ElevationPath();
	      map.addControl(measureProfil);
    	};

    	Gp.Services.getConfig({
		callbackSuffix : "",
		apiKey: "jhyvi0fgmnuxvfv0zjzorvdn",
		timeOut : 20000,
		onSuccess : createMap
    	});
    </script>
    </body>
</html>

Classes :

<html>
    <head>
	<link rel="stylesheet" href=".leaflet.css" />
        <script src="leaflet.js"></script>
        
        <script src="GpServices.js"></script>
        
	script src="WMTS.js"></script>
	
        <link rel="stylesheet" href="LayerSwitcher.css" />
        <script src="LayerSwitcher.js"></script>
        <link rel="stylesheet" href="Isocurve.css" />
        <script src="Isocurve.js"></script>
        <link rel="stylesheet" href="MousePosition.css" />
        <script src="MousePosition.js"></script>
        <link rel="stylesheet" href="Route.css" />
        <script src="Route.js"></script>
        <link rel="stylesheet" href="ReverseGeocode.css" />
        <script src="ReverseGeocode.js"></script>
        <link rel="stylesheet" href="SearchEngine.css" />
        <script src="SearchEngine.js"></script>
        <link rel="stylesheet" href="ElevationPath.css" />
        <script src="ElevationPath.js"></script>
    </head>
    <body>
        <div id="map"></div>
    	<script>
    	var createMap = function () {
	      // on cache l'image de chargement du Géoportail.
	      document.getElementById("map").style.backgroundImage = "none";

	      // Création de la map
	      var layer = new WMTS("http://wxs.ign.fr/jhyvi0fgmnuxvfv0zjzorvdn/geoportail/wmts", {
	          paramsNative : {
	              minZoom : 1,
	              maxZoom : 21
	          },
	          paramsWmts   : {
	              layer   : "ORTHOIMAGERY.ORTHOPHOTOS",
	              style   : "normal",
	              format  : "image/jpeg",
	              version : "1.0.0",
	              tilematrixset : "PM"
	          },
	          originators  : [],
	          legends      : [],
	          metadata     : [],
	          title        : "",
	          description  : "",
	          quicklookUrl : ""
	      });

	      var map  = L.map('map', {
		zoom : 2,
		center : L.latLng(48, 2)
	      });

	      layer.addTo(map);

	      var iso = new Isocurve();
	      map.addControl(iso);
	      var layerSwitcher = new LayerSwitcher();
	      map.addControl(layerSwitcher);
	      var mp = new MousePosition();
	      map.addControl(mp);
	      var route = new Route();
	      map.addControl(route);
	      var reverse = new ReverseGeocode();
	      map.addControl(reverse);
	      var search = new SearchEngine();
	      map.addControl(search);
	      var measureProfil = new ElevationPath();
	      map.addControl(measureProfil);
    	};

    	Gp.Services.getConfig({
		callbackSuffix : "",
		apiKey: "jhyvi0fgmnuxvfv0zjzorvdn",
		timeOut : 20000,
		onSuccess : createMap
    	});
    </script>
    </body>
</html>

Exécution de la page de test

$ ./node_modules/.bin/webpack-dev-server --config build/webpack/webpack.config.leaflet.modules --mode=development --https --open-page samples/index-leaflet-modules-map.html --content-base . --output-public-path '/dist/leaflet-modules/' --port 9001 --open

Demo

cf. https://ignf.github.io/geoportal-extensions/

intégrer les démos des modules sur la github pages en statique.

INFO

la publication dans le dépôt Leaflet impose d'avoir une demo. On peut donc maintenant fournir un lien !

TODO

finaliser les demos avec modules pour Leaflet / OpenLayers

Publication

script de publication (un répertoire / plugin) à exécuter via les github actions

préparation

$ cd build/scripts/release/
$ ./build-pack.sh -L

Les répertoires sont prêts pour la publication (+ tgz) :

$ cd build/scripts/release/
$ tree geoportail-extensions-leaflet-modules/
geoportal-extensions-leaflet-modules/
├── ElevationPath
├── ignf-geoportal-plugin-leaflet-elevationpath-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-isocurve-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-layers-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-layerswitcher-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-mouseposition-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-reversegeocode-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-route-2.2.7.tgz
├── ignf-geoportal-plugin-leaflet-searchengine-2.2.7.tgz
├── Isocurve
├── Layers
├── LayerSwitcher
├── MousePosition
├── ReverseGeocode
├── Route
└── SearchEngine

$ tree -L 2 geoportal-extensions-leaflet-modules/ElevationPath/
geoportal-extensions-leaflet-modules/ElevationPath/
├── dist
│   ├── ElevationPath.css
│   ├── ElevationPath.js
│   ├── ElevationPath-map.css
│   ├── ElevationPath-map.js
│   ├── ElevationPath-src.css
│   └── ElevationPath-src.js
├── LICENCE.md
├── package.json
├── README.md
└── src
   ├── Common
   │   ├── Controls
   │   │   ├── ElevationPathDOM.js
   │   │   └── ProfileElevationPathDOM.js
   │   └── Utils
   │       ├── AutoLoadConfig.js
   │       ├── CheckRightManagement.js
   │       ├── LoggerByDefault.js
   │       └── SelectorID.js
   └── Leaflet
       ├── Controls
       │   ├── ElevationPath.js
       │   └── Utils
       │       ├── IconDefault.js
       │       └── PositionFormater.js
       └── CSS
           └── Controls
               └── ElevationPath
                   └── GPelevationPathLeaflet.css

FIXME
trouver un namespace avec les infos : ignf / geoportal / plugin / leaflet
ex. @ignf-geoportal/plugin-leaflet-elevationpath

TODO

mettre en place des README personnalisés pour chaque widget.

ajouter un exemple

publication via npm

Le script nous fournit les répertoire de publication par widget.
Il nous reste donc à mettre en place ou modifier le job de publication des github actions.

TODO

opération à réaliser via les github actions

Guide d’intégration dans le dépôt Leaflet

cf. https://github.com/Leaflet/Leaflet/blob/main/PLUGIN-GUIDE.md

TODO

lister les actions obligatoires pour pouvoir intégrer nos modules dans le dépôts Leaflet (ex. demo)

lowzonenose · 2022-12-07T11:21:45Z

BUG

il semble avoir un bug sur l'utilisation du contrôle Isochrone : la requête ne passe pas car le point de saisie n'est pas correctement renvoyé au service !?

azarz · 2022-12-07T13:00:11Z

Beau travail :)

Pour le CSS, pour moi ce n'est pas rédhibitoire qu'il n'y en ait qu'un : il suffit d'avoir un lien statiques (cdnjs, raw.github) et de documenter que peu importe le widget, il faut linker le CSS dans le après le CSS leaflet et avant le CSS custom.

J'essaie de regarder le souci sur isochrone

… plugin de creation du manifest)

lowzonenose · 2022-12-09T16:21:23Z

RaF / Questions

@azarz / @lowzonenose

décision sur le workflow :

les modules sont ils automatiquement buildés et publiés lors des releases Leaflet ?
choisir un namespace des modules : ex. @ignf-geoportal/plugin-leaflet-elevationpath
ajouter le job de publication npm dans les github actions
copier les modules / activer les demos sur les modules sur la gh-page
comment gérer le fichier Common.css :

doit il être inclus dans chaque module ou est ce un module à part entière ?
doit on écrire un README spécifique à chaque module ou garde t on le README du projet ?
doit on fournir un exemple pour chaque module ?
doit on garder les répertoire de publication des modules sur le dépôt github ?

azarz · 2022-12-13T16:33:55Z

Mon avis sur les questions (aidé par https://github.com/Leaflet/Leaflet/blob/main/PLUGIN-GUIDE.md) :

RaF / Questions

@azarz / @lowzonenose

* [ ]  décision sur le **workflow** :
  > les modules sont ils automatiquement _buildés_ et _publiés_ lors des _releases_ Leaflet ?

Oui

* [ ]  choisir un **namespace** des modules : ex. `@ignf-geoportal/plugin-leaflet-elevationpath`

Les plugins Leaflet sont plutôt préfixés par leaflet (https://github.com/Leaflet/Leaflet/blob/main/PLUGIN-GUIDE.md#name)
Je pencherias donc plutôt sur du @ignf-geoportal/leaflet-elevationpath

* [ ]  comment gérer le fichier **Common.css** :
  > doit il être inclus dans chaque module ou est ce un module à part entière ?

Les 2 alternatives sont bien. Inclus dans chaque module permet de ne pas avoir à le linker à chaque fois, donc mon cœur penche plutôt pour cette solution.

* [ ]  doit on écrire un **README** spécifique à chaque module ou garde t on le **README** du projet ?

Oui, mais extrait à partir d'une sous-partie du readme du projet (exemple, ne prendre que le paragraphe https://github.com/IGNF/geoportal-extensions/blob/develop/doc/README-leaflet.md#profil-altim%C3%A9trique-le-long-dun-tra%C3%A7%C3%A9 pour le profil alti)
Je ne sais pas si c'est possible facilement...

* [ ]  doit on fournir un **exemple** pour chaque module ?

Oui

* [ ]  doit on garder les **répertoire de publication** des modules sur le dépôt _github_ ?

Oui (cf Turf.js)

elias75015 · 2023-04-19T15:51:39Z

@lowzonenose @azarz Petit point à prévoir sur le sujet pour expliciter le reste à faire, et le kifékoi

Gestion de Leaflet en package par modules

4fb1da7

lowzonenose requested a review from azarz December 5, 2022 08:37

Mise en place du script de préparation des publications

01572a2

lowzonenose added 3 commits December 8, 2022 16:50

Ajout des sources des modules dans les répertoire de publication (via…

243b923

… plugin de creation du manifest)

Fix sur la copie des sources

cc1685e

Merge branch 'develop' into feature/leaflet-modules

4439019

Preparation pour packaging modules

ad48565

Navigation Menu

Uh oh!

Gestion de Leaflet en package par modules - #344

Gestion de Leaflet en package par modules#344
lowzonenose wants to merge 6 commits into
developfrom
feature/leaflet-modules

lowzonenose commented Dec 5, 2022 •

edited

Loading

Uh oh!

lowzonenose commented Dec 7, 2022

Uh oh!

azarz commented Dec 7, 2022

Uh oh!

lowzonenose commented Dec 9, 2022 •

edited

Loading

Uh oh!

azarz commented Dec 13, 2022

RaF / Questions

Uh oh!

elias75015 commented Apr 19, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

lowzonenose commented Dec 5, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PLUGINS LEAFLET

Split en modules

taille des modules

carte des dépendances

Utilisation des modules dans une page web

Exécution de la page de test

Demo

Publication

préparation

publication via npm

Guide d’intégration dans le dépôt Leaflet

Uh oh!

lowzonenose commented Dec 7, 2022

Uh oh!

azarz commented Dec 7, 2022

Uh oh!

lowzonenose commented Dec 9, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

RaF / Questions

Uh oh!

azarz commented Dec 13, 2022

RaF / Questions

Uh oh!

elias75015 commented Apr 19, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lowzonenose commented Dec 5, 2022 •

edited

Loading

lowzonenose commented Dec 9, 2022 •

edited

Loading