empowerplan / epp-app

EmPowerPlan - App
GNU Affero General Public License v3.0
0 stars 0 forks source link

Settlement layer switch activates settlement 200m layer too #57

Closed nesnoj closed 7 months ago

nesnoj commented 7 months ago

Reproduce: Activate the settlements, both the settlements and the settlements 200m settlement layer is shown (the 200m button is deactivated though). Then switch the 200m settlement on and off and you get the correct settlements. :thinking:

I tested both with and without MVTs, same result.

henhuy commented 7 months ago

You are right. I implemented layer (de-)activation by activating all layers starting with the given layer name. Thus, pv_ground_criteria_settlementspv_ground_criteria_settlements triggers pv_ground_criteria_settlements_200m as well, but not the other way round. The reason I implemented it this way was to activate all cluster layers at once when triggering a cluster switch, as clusters constist of three layers: wind (single wind parks showing icon and clickable), wind_cluster (circles) and wind_cluster_count (number in circles). I will make a change, so that only given layer name is activated, and will check if cluster is involved and trigger related layers if needed.