Optimize setting line style

FacilMap · Mar 2, 2024 · b13025c · b13025c
1 parent 8d857d4
commit b13025c
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/leaflet/src/lines/lines-layer.ts b/leaflet/src/lines/lines-layer.ts
@@ -270,7 +270,13 @@ export default class LinesLayer extends FeatureGroup {
 		}
 
 		(this.linesById[line.id] as any).line = line;
-		this.linesById[line.id].setLatLngs(splitLatLngs).setStyle(style);
+
+		// Set style before setting coordinates, so a new line doesn't have to be rendered twice
+		if (Object.entries(style).some(([k, v]) => (this.linesById[line.id].realOptions as any)[k] !== v)) {
+			this.linesById[line.id].setStyle(style);
+		}
+
+		this.linesById[line.id].setLatLngs(splitLatLngs);
 
 		if (line.name && line.id != null) { // We don't want a popup for lines that we are drawing right now
 			const quoted = quoteHtml(line.name);