@geogirafe/lib-geoportal 1.2.0-dev.2870319621 → 1.2.0-dev.2873796427

package/components/userpreferences/component.js

@@ -15,12 +15,16 @@ export default class UserPreferencesComponent extends GirafeHTMLElement {
     styleUrls = null;
     template = () => {
         return uHtml `<style>
+.picker_wrapper.no_alpha .picker_alpha{display:none}.picker_wrapper.no_editor .picker_editor{z-index:-1;opacity:0;position:absolute}.picker_wrapper.no_cancel .picker_cancel{display:none}.layout_default.picker_wrapper{flex-flow:wrap;justify-content:space-between;align-items:stretch;width:25em;padding:.5em;font-size:10px;display:flex}.layout_default.picker_wrapper input,.layout_default.picker_wrapper button{font-size:1rem}.layout_default.picker_wrapper>*{margin:.5em}.layout_default.picker_wrapper:before{content:"";order:1;width:100%;height:0;display:block}.layout_default .picker_slider,.layout_default .picker_selector{padding:1em}.layout_default .picker_hue{width:100%}.layout_default .picker_sl{flex:auto}.layout_default .picker_sl:before{content:"";padding-bottom:100%;display:block}.layout_default .picker_editor{order:1;width:6.5rem}.layout_default .picker_editor input{width:100%;height:100%}.layout_default .picker_sample{flex:auto;order:1}.layout_default .picker_done,.layout_default .picker_cancel{order:1}.picker_wrapper{box-sizing:border-box;cursor:default;color:#444;pointer-events:auto;background:#f2f2f2;font-family:sans-serif;box-shadow:0 0 0 1px silver}.picker_wrapper:focus{outline:none}.picker_wrapper button,.picker_wrapper input{box-sizing:border-box;border:none;outline:none;box-shadow:0 0 0 1px silver}.picker_wrapper button:focus,.picker_wrapper button:active,.picker_wrapper input:focus,.picker_wrapper input:active{box-shadow:0 0 2px 1px #1e90ff}.picker_wrapper button{cursor:pointer;background-color:#f5f5f5;background-image:linear-gradient(#0000,#dcdcdc);padding:.4em .6em}.picker_wrapper button:active{background-image:linear-gradient(#dcdcdc,#0000)}.picker_wrapper button:hover{background-color:#fff}.picker_selector{z-index:1;cursor:pointer;background:currentColor;border:2px solid #fff;border-radius:100%;display:block;position:absolute;-webkit-transform:translate(-50%,-50%);transform:translate(-50%,-50%);box-shadow:0 0 3px 1px #67b9ff}.picker_slider .picker_selector{border-radius:2px}.picker_hue{background-image:linear-gradient(90deg,red,#ff0,#0f0,#0ff,#00f,#f0f,red);position:relative;box-shadow:0 0 0 1px silver}.picker_sl{background-image:linear-gradient(#fff,#fff0 50%),linear-gradient(#0000 50%,#000),linear-gradient(90deg,gray,#80808000);position:relative;box-shadow:0 0 0 1px silver}.picker_alpha,.picker_sample{background:linear-gradient(45deg,#d3d3d3 25%,#0000 25% 75%,#d3d3d3 75%) 0 0/2em 2em,linear-gradient(45deg,#d3d3d3 25%,#fff 25% 75%,#d3d3d3 75%) 1em 1em/2em 2em;position:relative;box-shadow:0 0 0 1px silver}.picker_alpha .picker_selector,.picker_sample .picker_selector{background:0 0}.picker_editor input{padding:.2em .4em;font-family:monospace}.picker_sample:before{content:"";background:currentColor;width:100%;height:100%;display:block;position:absolute}.picker_arrow{z-index:-1;position:absolute}.picker_wrapper.popup{z-index:2;margin:1.5em;position:absolute}.picker_wrapper.popup,.picker_wrapper.popup .picker_arrow:before,.picker_wrapper.popup .picker_arrow:after{background:#f2f2f2;box-shadow:0 0 10px 1px #0006}.picker_wrapper.popup .picker_arrow{width:3em;height:3em;margin:0}.picker_wrapper.popup .picker_arrow:before,.picker_wrapper.popup .picker_arrow:after{content:"";z-index:-99;display:block;position:absolute;top:0;left:0}.picker_wrapper.popup .picker_arrow:before{-webkit-transform-origin:0 100%;transform-origin:0 100%;width:100%;height:100%;-webkit-transform:skew(45deg);transform:skew(45deg)}.picker_wrapper.popup .picker_arrow:after{width:150%;height:150%;box-shadow:none}.popup.popup_top{bottom:100%;left:0}.popup.popup_top .picker_arrow{bottom:0;left:0;-webkit-transform:rotate(-90deg);transform:rotate(-90deg)}.popup.popup_bottom{top:100%;left:0}.popup.popup_bottom .picker_arrow{top:0;left:0;-webkit-transform:rotate(90deg)scaleY(-1);transform:rotate(90deg)scaleY(-1)}.popup.popup_left{top:0;right:100%}.popup.popup_left .picker_arrow{top:0;right:0;-webkit-transform:scaleX(-1);transform:scaleX(-1)}.popup.popup_right{top:0;left:100%}.popup.popup_right .picker_arrow{top:0;left:0}
+</style><style>
+.picker_wrapper,.picker_arrow:after,.picker_arrow:before{background-color:var(--bkg-color)!important}.picker_wrapper,.picker_arrow:before{box-shadow:0 0 10px 1px var(--text-color-grad1)!important}.picker_wrapper.fixedPosition{transform:translate(-260px)}.picker_arrow.fixedPosition{transform:translate(200px)rotate(90deg)scaleX(-1)!important}
+</style><style>
 .hidden{display:none!important}.gg-rotate90{transform:rotate(90deg)}.gg-rotate180{transform:rotate(180deg)}.gg-rotate270{transform:rotate(270deg)}img{filter:var(--svg-filter)}img.legend-image{filter:var(--svg-map-filter);background:var(--svg-legend-bkg)}div{scrollbar-width:thin}a,a:visited{color:var(--link-color)}@keyframes spin{0%{transform:rotate(0)}to{transform:rotate(360deg)}}@keyframes spin-wait{0%{transform:rotate(0)}7%{transform:rotate(360deg)}to{transform:rotate(360deg)}}.gg-spin{animation-name:spin;animation-duration:2s;animation-timing-function:linear;animation-iteration-count:infinite}.gg-spin-wait{animation-name:spin-wait;animation-duration:10s;animation-timing-function:linear;animation-iteration-count:infinite}::-webkit-scrollbar{width:5px}::-webkit-scrollbar-thumb{background:#999}button,input,select,textarea{font:inherit}.gg-button,.gg-select,.gg-input,.gg-textarea{background-color:var(--bkg-color);color:var(--text-color);border:var(--app-standard-border);box-sizing:border-box;cursor:pointer;border-radius:3px;outline:0;margin:0;padding:0 0 0 .5rem;display:inline-block}.gg-label{background-color:var(--bkg-color);color:var(--text-color);border:none;align-items:center;margin:0;padding:0;display:flex}.gg-button,.gg-select,.gg-input,.gg-label{min-height:calc(var(--app-standard-height) / 1.5)}.gg-textarea{max-height:initial;resize:vertical;height:6rem;padding:.5rem;line-height:1.3rem}.gg-input{cursor:text}.gg-checkbox{accent-color:var(--text-color);cursor:pointer;width:1.2rem}.gg-range{accent-color:var(--text-color)}.gg-button{padding:0 .5rem}.gg-button.active{border:solid 1px var(--text-color-grad1);background-color:var(--bkg-color-grad2)}.gg-button:disabled{background-color:var(--bkg-color-grad1);color:var(--text-color-grad2);cursor:not-allowed;border:none}.gg-button:disabled img{filter:opacity(.6)}.gg-input:disabled,.gg-select:disabled,.gg-textarea:disabled{background-color:var(--bkg-color-grad1);color:var(--text-color-grad2);cursor:not-allowed}.gg-button>img{vertical-align:middle}.gg-icon-button{color:var(--text-color);cursor:pointer;background-color:#0000;border:none;flex-direction:column;justify-content:center;align-items:center;padding:0;display:flex}.gg-icon{justify-content:center;align-items:center;display:flex}.gg-big,.gg-big-withtext{min-width:var(--app-standard-height);min-height:var(--app-standard-height);max-height:var(--app-standard-height)}.gg-big img,.gg-big-withtext img{width:calc(var(--app-standard-height) - 1.5rem);margin:0}.gg-big-withtext span{font-variant:small-caps;padding:0 1rem;font-size:.9rem}.gg-medium,.gg-medium-withtext{min-width:calc(var(--app-standard-height) / 1.2);min-height:calc(var(--app-standard-height) / 1.2);max-height:calc(var(--app-standard-height) / 1.2);flex-direction:row}.gg-medium img{width:calc(var(--app-standard-height) / 2.4);margin:0}.gg-medium-withtext img{width:calc(var(--app-standard-height) / 2.4);margin-left:.5rem}.gg-medium-withtext span{padding:0 1rem 0 .5rem;font-size:.9rem}.gg-small,.gg-small-withtext{min-width:calc(var(--app-standard-height) / 2);min-height:calc(var(--app-standard-height) / 2);max-height:calc(var(--app-standard-height) / 2);flex-direction:row}.gg-small img{width:calc(var(--app-standard-height) / 3);margin:0}.gg-small-withtext img{width:calc(var(--app-standard-height) / 3);margin-left:.5rem}.gg-small-withtext span{padding:0 .5rem 0 .3rem;font-size:.9rem}.gg-button:hover:not(:disabled),.gg-select:hover:not(:disabled),.gg-input:hover:not(:disabled),.gg-textarea:hover:not(:disabled),.gg-icon-button:hover:not(:disabled){background-color:var(--bkg-color-grad1)}.gg-opacity{opacity:.5}.gg-opacity:hover{opacity:1;background-color:#0000}.gg-tabs{cursor:pointer;grid-auto-flow:column;padding-bottom:1rem;font-size:1rem;display:grid}.gg-tab{border:none;border-bottom:var(--app-standard-border);cursor:pointer;color:var(--text-color);background:0 0;padding:.5rem}.gg-tab.active{border-bottom:solid 1px var(--text-color)}.girafe-button-big,.girafe-button-large,.girafe-button-small,.girafe-button-tiny{color:var(--text-color);background-color:#0000;border:none;flex-direction:column;display:flex}.girafe-button-big:hover,.girafe-button-large:hover,.girafe-button-small:hover,.girafe-button-tiny:hover{background-color:var(--bkg-color-grad1);cursor:pointer}.girafe-button-big.dark,.girafe-button-large.dark,.girafe-button-small.dark,.girafe-button-tiny.dark{background-color:var(--bkg-color);filter:invert()}.girafe-button-big{width:var(--app-standard-height);height:var(--app-standard-height);align-items:center;padding:1rem}.girafe-button-big img{overflow:hidden}.girafe-button-large{flex-direction:row}.girafe-button-large img{height:2rem;margin:.3rem}.girafe-button-large span{height:2rem;margin:.3rem;line-height:2rem}.girafe-button-small{min-width:calc(var(--app-standard-height) / 2);height:calc(var(--app-standard-height) / 2);align-items:center;padding:.5rem}.girafe-button-small img{overflow:hidden}.girafe-button-small span{text-align:left;text-overflow:ellipsis;width:100%;overflow:hidden}.girafe-button-tiny{align-items:center;width:1rem;height:1rem;padding:0}.girafe-button-tiny img{overflow:hidden}.girafe-onboarding-theme{background-color:var(--bkg-color)!important;color:var(--text-color)!important}.girafe-onboarding-theme button{background-color:var(--bkg-color)!important;color:var(--text-color)!important;text-shadow:none!important}.girafe-onboarding-theme button.driver-popover-close-btn{z-index:10000}.tippy-box{background-color:var(--bkg-color)!important;border:var(--app-standard-border)!important}.tippy-content{color:var(--text-color)!important}.tippy-arrow{color:var(--bkg-color)!important}.tippy-box[data-theme~=error]{background-color:var(--error-color);white-space:pre-line}div.tippy-box[data-theme=image-popup]{border:var(--app-standard-border)!important;& div.tippy-content{background-color:var(--bkg-color)!important;& img{max-width:20rem;max-height:20rem}}& div.tippy-arrow{color:var(--bkg-color)!important}}
 </style><style>
 #content{background:var(--bkg-color);color:var(--text-color);margin:0;padding:1rem}.preference-group{padding-bottom:1rem}.preference-group-title{padding-bottom:10px;font-weight:700}.preference-row{flex-direction:column;padding-bottom:8px;display:flex}.with-checkbox,.with-colorpicker{flex-direction:row;justify-content:space-between;align-items:center;height:2rem}.colorpicker{border:solid 1px var(--text-color);cursor:pointer;border-radius:3px;width:1.5rem;height:1.5rem}#reset-all{justify-self:end}#footer{margin-top:2rem}
 </style>
 <style>${this.customStyle}</style>
-<link rel="stylesheet" href="lib/vanilla-picker/vanilla-picker.csp.css"><link rel="stylesheet" href="styles/girafecolorpicker.css"><div id="panel"><div id="content">${this.preferenceGroups.map(group => uHtml `<div class="preference-group"><div class="preference-group-title" i18n="${group}">${group}</div>${this.getPreferencesByGroup(group).map(([preference, key]) => uHtmlFor(preference, key) ` ${preference.uiElement === 'select' ? uHtml `<div class="preference-row" hidden="${!preference.visible}"><label for="${key}" class="gg-label" i18n="${key}">${key}</label> <select id="${key}" class="gg-select" onchange="${(evt) => this.onSelectOption(key, evt)}" onmousedown="${() => this.onOpenSelectOption(key)}">${preference.options.map(option => uHtml `<option i18n="${option.label}" ?selected="${preference.currentValue === option.value}" value="${option.label}">${option.label}</option>`)}</select></div>` : uHtml ``} ${preference.uiElement === 'checkbox' ? uHtml `<div class="preference-row with-checkbox" hidden="${!preference.visible}"><label for="${key}" i18n="${key}">${key}</label> <input id="${key}" type="checkbox" class="gg-checkbox" ?checked="${preference.currentValue}" onchange="${() => this.changePreference(key, !preference.currentValue)}"></div>` : uHtml ``} ${preference.uiElement === 'color' ? uHtml `<div class="preference-row with-colorpicker" hidden="${!preference.visible}"><label for="${key}" i18n="${key}">${key}</label> <button class="colorpicker" id="${key}"></button></div>` : uHtml ``} `)}</div>`)}<div id="footer"><button id="reset-all" class="gg-button" tip="Reset all preferences back to the default value" i18n="Reset All" onclick="${() => this.onResetAll()}">Reset All</button></div></div></div>
+<div id="panel"><div id="content">${this.preferenceGroups.map(group => uHtml `<div class="preference-group"><div class="preference-group-title" i18n="${group}">${group}</div>${this.getPreferencesByGroup(group).map(([preference, key]) => uHtmlFor(preference, key) ` ${preference.uiElement === 'select' ? uHtml `<div class="preference-row" hidden="${!preference.visible}"><label for="${key}" class="gg-label" i18n="${key}">${key}</label> <select id="${key}" class="gg-select" onchange="${(evt) => this.onSelectOption(key, evt)}" onmousedown="${() => this.onOpenSelectOption(key)}">${preference.options.map(option => uHtml `<option i18n="${option.label}" ?selected="${preference.currentValue === option.value}" value="${option.label}">${option.label}</option>`)}</select></div>` : uHtml ``} ${preference.uiElement === 'checkbox' ? uHtml `<div class="preference-row with-checkbox" hidden="${!preference.visible}"><label for="${key}" i18n="${key}">${key}</label> <input id="${key}" type="checkbox" class="gg-checkbox" ?checked="${preference.currentValue}" onchange="${() => this.changePreference(key, !preference.currentValue)}"></div>` : uHtml ``} ${preference.uiElement === 'color' ? uHtml `<div class="preference-row with-colorpicker" hidden="${!preference.visible}"><label for="${key}" i18n="${key}">${key}</label> <button class="colorpicker" id="${key}"></button></div>` : uHtml ``} `)}</div>`)}<div id="footer"><button id="reset-all" class="gg-button" tip="Reset all preferences back to the default value" i18n="Reset All" onclick="${() => this.onResetAll()}">Reset All</button></div></div></div>
 ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
     };
     isPanelVisible = false;

package/models/layers/grouplayer.js

@@ -21,7 +21,7 @@ class GroupLayer extends BaseLayer {
     children = [];
     constructor(id, name, order, options) {
         super(id, name, order, options);
-        this.isExpanded = options?.isDefaultExpanded || false;
+        this.isExpanded = options?.isDefaultExpanded ?? false;
         this.isExclusiveGroup = options?.isExclusiveGroup ?? false;
         this._isMixed = options?.isMixed;
         this.timeOptions = options?.time;

package/models/layers/themelayer.js

@@ -17,7 +17,7 @@ class ThemeLayer extends BaseLayer {
     children = [];
     constructor(id, name, order, icon, options) {
         super(id, name, order, options);
-        this.isExpanded = options?.isDefaultExpanded || true;
+        this.isExpanded = options?.isDefaultExpanded ?? true;
         this.isExclusiveTheme = options?.isExclusiveTheme ?? false;
         this.icon = icon;
     }

package/models/xmlTypes.js

@@ -40,7 +40,7 @@ export const xmlTypesStrList = [
     ...xmlGeometryTypesStrList
 ];
 export function isStringNumeric(str) {
-    return str.trim().length > 0 && !isNaN(Number(str)); // ensure strings of whitespace fail or mixed chars and numbers like '2n'
+    return str.trim().length > 0 && !Number.isNaN(Number(str)); // ensure strings of whitespace fail or mixed chars and numbers like '2n'
 }
 export function isString(attributeType) {
     return xmlStringTypesStrList.includes(attributeType);

package/package.json

@@ -5,7 +5,7 @@
     "name": "GeoGirafe PSC",
     "url": "https://doc.geogirafe.org"
   },
-  "version": "1.2.0-dev.2870319621",
+  "version": "1.2.0-dev.2873796427",
   "type": "module",
   "engines": {
     "node": ">=20.19.0"

package/templates/iframe.html

@@ -28,44 +28,10 @@
     <link rel="stylesheet" href="styles/common.css" />
     <link rel="stylesheet" href="styles/iframe.css" />
 
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/tippy.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/backdrop.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/border.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/svg-arrow.css" />
-    <noscript>
-      <link rel="stylesheet" href="lib/tippy.js/tippy.css" />
-      <link rel="stylesheet" href="lib/tippy.js/backdrop.css" />
-      <link rel="stylesheet" href="lib/tippy.js/border.css" />
-      <link rel="stylesheet" href="lib/tippy.js/svg-arrow.css" />
-    </noscript>
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/tippy.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/backdrop.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/border.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/svg-arrow.css" />
 
     <script type="module" src="src/main.iframe.ts"></script>
     <title>GeoGirafe iFrame</title>

package/templates/index.html

@@ -33,44 +33,10 @@
     <link rel="stylesheet" href="styles/common.css" />
     <link rel="stylesheet" href="styles/index.css" />
 
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/tippy.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/backdrop.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/border.css" />
-    <link
-      rel="preload"
-      as="style"
-      onload="
-        this.onload = null;
-        this.rel = 'stylesheet';
-      "
-      href="lib/tippy.js/svg-arrow.css" />
-    <noscript>
-      <link rel="stylesheet" href="lib/tippy.js/tippy.css" />
-      <link rel="stylesheet" href="lib/tippy.js/backdrop.css" />
-      <link rel="stylesheet" href="lib/tippy.js/border.css" />
-      <link rel="stylesheet" href="lib/tippy.js/svg-arrow.css" />
-    </noscript>
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/tippy.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/backdrop.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/border.css" />
+    <link rel="stylesheet" href="node_modules/tippy.js/dist/svg-arrow.css" />
 
     <script type="module" src="src/main.ts"></script>
     <title>GeoGirafe</title>

package/templates/public/about.json

@@ -1 +1 @@
-{"version":"1.2.0-dev.2870319621", "build":"2870319621", "date":"22/09/2026"}
+{"version":"1.2.0-dev.2873796427", "build":"2873796427", "date":"23/09/2026"}

package/templates/vite.config.js

@@ -141,10 +141,7 @@ export default defineConfig(({ command, mode }) => {
           { src: `${geogirafeSource}/tools/sw/*.js`, dest: '' },
           { src: `${geogirafeSource}/styles/*.css`, dest: 'styles/' },
           { src: `${geogirafeSource}/assets/*`, dest: '' },
-          { src: `${geogirafeSource}/tools/auth/silentlogincallback.html`, dest: '' },
-          { src: 'node_modules/tabulator-tables/dist/css/tabulator.min.css', dest: 'lib/tabulator-tables/' },
-          { src: 'node_modules/tippy.js/dist/*.css', dest: 'lib/tippy.js/' },
-          { src: 'node_modules/vanilla-picker/dist/*.css', dest: 'lib/vanilla-picker/' }
+          { src: `${geogirafeSource}/tools/auth/silentlogincallback.html`, dest: '' }
         ]
       }),
       // This second usage of viteStaticCopy is dedicated to the library.

package/tools/ordering/orderingmanager.d.ts

@@ -2,6 +2,7 @@ import GirafeSingleton from '../../base/GirafeSingleton.js';
 export declare const LayerTreeStartOrder = 1000;
 export default class OrderingManager extends GirafeSingleton {
     private timeoutId?;
+    private isReordering;
     private get state();
     initializeSingleton(): void;
     private registerEvents;

package/tools/ordering/orderingmanager.js

@@ -5,6 +5,7 @@ import ThemeLayer from '../../models/layers/themelayer.js';
 export const LayerTreeStartOrder = 1000;
 export default class OrderingManager extends GirafeSingleton {
     timeoutId;
+    isReordering = false;
     get state() {
         return this.context.stateManager.state;
     }
@@ -14,12 +15,17 @@ export default class OrderingManager extends GirafeSingleton {
     registerEvents() {
         this.context.stateManager.subscribe('layers.layersList', (oldLayers, newLayers) => this.onLayerListChanged(oldLayers, newLayers));
         this.context.stateManager.subscribe(/layers\.layersList\..*\.children/, (oldLayers, newLayers) => this.onLayerListChanged(oldLayers, newLayers));
-        this.context.stateManager.subscribe(/layers\.layersList\..*\.order/, () => this.reorderLayers());
+        this.context.stateManager.subscribe(/layers\.layersList\..*\.order/, () => {
+            if (!this.isReordering) {
+                this.reorderLayers();
+            }
+        });
     }
     onLayerListChanged(oldLayers, newLayers) {
         // Need to reorder only if new layers were added.
-        const addedLayers = newLayers.filter((newChild) => !oldLayers.find((oldChild) => oldChild.treeItemId === newChild.treeItemId));
-        if (addedLayers.length > 0) {
+        const oldTreeItemIds = new Set(oldLayers.map((oldChild) => oldChild.treeItemId));
+        const hasAddedLayers = newLayers.some((newChild) => !oldTreeItemIds.has(newChild.treeItemId));
+        if (hasAddedLayers) {
             this.reorderLayers();
         }
     }
@@ -35,16 +41,37 @@ export default class OrderingManager extends GirafeSingleton {
             clearTimeout(this.timeoutId);
         }
         this.timeoutId = setTimeout(() => {
-            const orderedLayers = this.getSortedLayers(this.state.layers.layersList);
-            const counter = { index: LayerTreeStartOrder };
-            this.context.stateManager.batchChanges(() => this.reorderLayersRecursively(orderedLayers, counter));
+            this.timeoutId = undefined;
+            this.isReordering = true;
+            const wasRenderEnabled = this.state.treeview.renderEnabled;
+            if (wasRenderEnabled) {
+                this.state.treeview.renderEnabled = false;
+            }
+            try {
+                const orderedLayers = this.getSortedLayers(this.state.layers.layersList);
+                const counter = { index: LayerTreeStartOrder };
+                this.context.stateManager.batchChanges(() => this.reorderLayersRecursively(orderedLayers, counter));
+            }
+            finally {
+                if (wasRenderEnabled) {
+                    this.state.treeview.renderEnabled = true;
+                }
+                this.isReordering = false;
+            }
         });
     }
     reorderLayersRecursively(orderedLayers, counter) {
         // Traverse the layertree in depth first to renumber all the elements
         for (const layer of orderedLayers) {
-            console.debug(`Setting layer ${layer.name} to order=${counter.index}`);
-            layer.order = counter.index++;
+            // NOTE REG : Setting a variable to the very same value is managed by brain in theory,
+            // and no event will be sent by brain is the value is identical.
+            // But even if the call to a proxy has a small cost, here during a reorder, it can be called many times
+            // Because we reorder the whole treeview, which can be very big.
+            // Therefore, it is more performant here to add this test before setting the variable to the same value.
+            const nextOrder = counter.index++;
+            if (layer.order !== nextOrder) {
+                layer.order = nextOrder;
+            }
             if (layer instanceof GroupLayer || layer instanceof ThemeLayer) {
                 const orderedChilds = this.getSortedLayers(layer.children);
                 this.reorderLayersRecursively(orderedChilds, counter);

package/tools/resizewindow.js

@@ -108,12 +108,12 @@ export default class ResizeWindow {
         this.width = dimension.width;
         this.height = dimension.height;
         const css = getComputedStyle(this.host);
-        this.minHeight = parseInt(css.minHeight) || 0;
-        this.maxHeight = parseInt(css.maxHeight) || Infinity;
-        this.minWidth = parseInt(css.minWidth) || 0;
-        this.maxWidth = parseInt(css.maxWidth) || Infinity;
-        this.top = parseInt(css.top) || 0;
-        this.left = parseInt(css.left) || 0;
+        this.minHeight = Number.parseInt(css.minHeight) || 0;
+        this.maxHeight = Number.parseInt(css.maxHeight) || Infinity;
+        this.minWidth = Number.parseInt(css.minWidth) || 0;
+        this.maxWidth = Number.parseInt(css.maxWidth) || Infinity;
+        this.top = Number.parseInt(css.top) || 0;
+        this.left = Number.parseInt(css.left) || 0;
         // Call a function whenever the cursor moves.
         document.onmousemove = (dragEvent) => this.resize(dragEvent, direction);
         // Detach events.

package/tools/share/serializers/globeserializer.d.ts

@@ -7,6 +7,9 @@ export default class GlobeSerializer implements IBrainSerializer<GlobeState> {
     private get globe();
     brainSerialize(globe: GlobeState): string;
     brainDeserialize(str: string): void;
-    /** Applies the layout a display mode of the previous versions corresponds to. */
+    /**
+     * Applies the layout a display mode of the previous versions corresponds to.
+     * TODO REG: Remove the legacy code when not used any more
+     * */
     private deserializeLegacyDisplay;
 }

package/tools/share/serializers/globeserializer.js

@@ -39,7 +39,10 @@ export default class GlobeSerializer {
         globe.loaded = false;
         this.deserializeLegacyDisplay(parsedGlobe.display);
     }
-    /** Applies the layout a display mode of the previous versions corresponds to. */
+    /**
+     * Applies the layout a display mode of the previous versions corresponds to.
+     * TODO REG: Remove the legacy code when not used any more
+     * */
     deserializeLegacyDisplay(display) {
         if (display === undefined) {
             return;

package/tools/state/brain/brain.d.ts

@@ -54,6 +54,7 @@ export default class Brain<T extends Record<string | symbol, any>> {
     private getOrCreateProxyForValue;
     private cleanProxyForValue;
     private recalculateChildrenForArray;
+    private cleanPreviousChildren;
     private updateChildPathsRecursively;
     private handleGetVirtualProperties;
     private handleGetIgnored;

package/tools/state/brain/brain.js

@@ -84,7 +84,7 @@ export default class Brain {
             parents: parents.map((p) => p.__brainId).join('|')
         });
         if (this.mergeMinimalPathsCache.has(cacheKey)) {
-            return this.mergeMinimalPathsCache.get(cacheKey);
+            return [...this.mergeMinimalPathsCache.get(cacheKey)];
         }
         const minimalPaths = new Set();
         const parentsPaths = [];
@@ -108,7 +108,7 @@ export default class Brain {
         }
         const returnValue = Array.from(minimalPaths);
         this.mergeMinimalPathsCache.set(cacheKey, returnValue);
-        return returnValue;
+        return [...returnValue];
     }
     isRightParent(candidate, parentsPaths) {
         if (parentsPaths.length === 0 || parentsPaths[0].length === 0) {
@@ -193,37 +193,48 @@ export default class Brain {
         // Recursively update child paths for consistency
         this.updateChildPathsRecursively(valueProxy, valueProxy.__brainFullPaths);
     }
-    recalculateChildrenForArray(proxy, target, oldValue) {
+    recalculateChildrenForArray(proxy, target, _oldValue) {
         if (!Array.isArray(target)) {
             throw new TypeError('This method is only for arrays');
         }
-        // Clean previous childs
-        proxy.__brainChildren.clear();
+        const currentChildProxies = new Set();
+        // Save the previous children in a temporary Set, and clean the children
+        // (We use this for performance reason : a Set is a lot more performant to access)
+        const previousChildren = new Map(proxy.__brainChildren);
+        const previousChildProxies = new Set(previousChildren.values());
+        this.cleanPreviousChildren(previousChildren, proxy);
         for (let i = 0; i < target.length; ++i) {
             const childValue = target[i];
             const childPaths = proxy.__brainFullPaths.map((path) => this.getFullPath(path, i.toString()));
-            const childOldValue = oldValue.find((oldChild) => areEqual(oldChild, childValue));
-            if (childOldValue) {
-                const oldChildIndex = oldValue.indexOf(childOldValue);
-                const oldChildPaths = proxy.__brainFullPaths.map((path) => this.getFullPath(path, oldChildIndex));
-                this.cleanProxyForValue(proxy, i.toString(), childValue, oldChildPaths);
-            }
             const childProxy = this.getOrCreateProxyForValue(proxy, i.toString(), childValue, childPaths);
             if (childProxy) {
                 proxy.__brainChildren.set(i.toString(), childProxy);
+                currentChildProxies.add(childProxy);
             }
-            if (oldValue && oldValue.length > target.length) {
-                for (let i = target.length; i < oldValue.length; i++) {
-                    const childPaths = proxy.__brainFullPaths.map((path) => this.getFullPath(path, i.toString()));
-                    this.cleanProxyForValue(proxy, i.toString(), oldValue[i], childPaths);
+        }
+        for (const childProxy of previousChildProxies) {
+            if (!currentChildProxies.has(childProxy)) {
+                const parents = childProxy.__brainParents;
+                const index = parents.indexOf(proxy);
+                if (index >= 0) {
+                    parents.splice(index, 1);
                 }
             }
-            for (const [key, childProxy] of proxy.__brainChildren.entries()) {
-                const childPaths = proxy.__brainFullPaths.map((path) => this.getFullPath(path, key));
-                this.proxyToFullPaths.set(childProxy, childPaths);
-                this.updateChildPathsRecursively(childProxy, childPaths);
+            this.updateChildPathsRecursively(childProxy, childProxy.__brainFullPaths);
+        }
+    }
+    cleanPreviousChildren(previousChildren, proxy) {
+        for (const [key, childProxy] of previousChildren.entries()) {
+            const fullPaths = childProxy.__brainFullPaths;
+            const childFullPaths = proxy.__brainFullPaths.map((path) => this.getFullPath(path, key));
+            for (const childFullPath of childFullPaths) {
+                const index = fullPaths.indexOf(childFullPath);
+                if (index >= 0) {
+                    fullPaths.splice(index, 1);
+                }
             }
         }
+        proxy.__brainChildren.clear();
     }
     updateChildPathsRecursively(proxy, parentPaths, visited = new Set()) {
         if (visited.has(proxy)) {

package/tools/time/layertimeformatter.js

@@ -152,7 +152,7 @@ class LayerTimeFormatter {
         return temporalFilter;
     }
     isValidDateString(dateString) {
-        return !isNaN(Date.parse(dateString));
+        return !Number.isNaN(Date.parse(dateString));
     }
     isWithinRange(date) {
         return date >= this.minValue && date <= this.maxValue;