From 9c5dd9efbf6fc7abe56df358b3f04bf3dd9999a4 Mon Sep 17 00:00:00 2001 From: Anton Khorev Date: Wed, 30 Apr 2025 02:49:36 +0300 Subject: [PATCH] Move creating changeset rectangles to bbox layers --- .../index/history-changesets-layer.js | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/app/assets/javascripts/index/history-changesets-layer.js b/app/assets/javascripts/index/history-changesets-layer.js index 6cdd096dd..be3f9ad1e 100644 --- a/app/assets/javascripts/index/history-changesets-layer.js +++ b/app/assets/javascripts/index/history-changesets-layer.js @@ -3,6 +3,13 @@ OSM.HistoryChangesetBboxLayer = L.FeatureGroup.extend({ return layer.id; }, + addChangesetLayer: function (changeset) { + const style = this._getChangesetStyle(changeset); + const rectangle = L.rectangle(changeset.bounds, style); + rectangle.id = changeset.id; + return this.addLayer(rectangle); + }, + updateChangesetLayerBounds: function (changeset) { this.getLayer(changeset.id)?.setBounds(changeset.bounds); }, @@ -119,15 +126,11 @@ OSM.HistoryChangesetsLayer = L.FeatureGroup.extend({ } for (const changeset of this._changesets.values()) { - const rect = L.rectangle(changeset.bounds, this._areaLayer._getChangesetStyle(changeset)); - rect.id = changeset.id; - rect.addTo(this._areaLayer); + this._areaLayer.addChangesetLayer(changeset); } for (const changeset of this._changesets.values()) { - const rect = L.rectangle(changeset.bounds, this._borderLayer._getChangesetStyle(changeset)); - rect.id = changeset.id; - rect.addTo(this._borderLayer); + this._borderLayer.addChangesetLayer(changeset); } }, @@ -136,9 +139,7 @@ OSM.HistoryChangesetsLayer = L.FeatureGroup.extend({ if (!changeset) return; if (state) { - const highlightRect = L.rectangle(changeset.bounds, this._highlightLayer._getChangesetStyle(changeset)); - highlightRect.id = id; - this._highlightLayer.addLayer(highlightRect); + this._highlightLayer.addChangesetLayer(changeset); } else { this._highlightLayer.removeLayer(id); } -- 2.39.5