Fix cluster layer #789
Fix cluster layer #789
Conversation
viatkinviatkin
commented
Jul 25, 2023
viatkinviatkin
commented
- fixed labels visibility on features load
- fixed _leafletObject getter for L.MarkerClusterGroup
- fixed L.MarkerClusterGroup layer crud
- fixed side-by-side for L.MarkerClusterGroup
- refactored
…xberry-gis into fix_clusterLayer_2507
| return; | ||
| } | ||
|
|
||
| if (this.get('clusterize')) { |
There was a problem hiding this comment.
Нужны комментарии
Что есть _leafletObject и что есть _originalVectorLayer
и что в каком случае испольуется
Возможно стоит переназвать метод
и Почему тут мы решили проверять не через instanceof L.MarkerClusterGroup ?
There was a problem hiding this comment.
- По мне логиченее сменить название
_originalVectorLayer->_leafletObject, чтобы понимать, что это за ссылка такая: вложенный _leafletObject в _leafletObject, верхний уровень L.MarkerClusterGroup, внутренний L.Marker - Исправил проверку кластеризованного слоя в
returnLeafletObject()на instanceOf - Добавил комментарии
| @@ -200,6 +215,9 @@ export default BaseLayer.extend({ | |||
|
|
|||
| if (this.get('labelSettings.signMapObjects')) { | |||
| this._addLabelsToLeafletContainer(layers, leafletObject); | |||
| if (this.get('_leafletObject') instanceof L.MarkerClusterGroup) { | |||
There was a problem hiding this comment.
тут тоже нужны комментарии. я не понимаю зачем это
There was a problem hiding this comment.
Иногда featuresProcessCallBack добавляет на карту надписи объектов слоя, который находится внутри кластера, необходимо почистить надписи обработчиком кластерного слоя
- на картинке видно, что на событие зума карты триггернулся featuresProcessCallBack (подгрузилось 0 объектов) внутреннего _originalVectorLayer,
- отработал метод _addLabelsToLeafletContainer, который добавил на карту все надписи внутреннего _originalVectorLayer
- а обработчик надписей для L.MarkerClusterGroup не триггернулся, поскольку он работает только на событие
layeraddвнутреннего слоя
There was a problem hiding this comment.
Добавил комментарии
|
Kudos, SonarCloud Quality Gate passed!
|
