ipyleaflet icon indicating copy to clipboard operation
ipyleaflet copied to clipboard

Add subitems to layers (either layers or controls or a mixture of both)

Open HaudinFlorence opened this issue 3 years ago • 1 comments
trafficstars

Add subitems to layers (either layers or controls or a mixture of both).

HaudinFlorence avatar Jul 01 '22 16:07 HaudinFlorence

Could you rebase on master?

Thanks for the review. Rebase has been made.

HaudinFlorence avatar Aug 10 '22 17:08 HaudinFlorence

It would be great to add a galata test for this :)

A notebook has been added in ui-tests/notebooks in e0b19ac, to test adding subitems in layers and check that they correctly display.

HaudinFlorence avatar Sep 06 '22 15:09 HaudinFlorence

please update galata references

HaudinFlorence avatar Sep 07 '22 09:09 HaudinFlorence

Please update galata reference

HaudinFlorence avatar Sep 08 '22 07:09 HaudinFlorence

please update galata references

HaudinFlorence avatar Sep 12 '22 11:09 HaudinFlorence

please update galata references

HaudinFlorence avatar Sep 12 '22 13:09 HaudinFlorence

The CI tests are now green. Is there other things I may add or check ?

HaudinFlorence avatar Sep 15 '22 09:09 HaudinFlorence

Thanks! I will give the PR a try. I suspect there is a bug when you create the map in a first Notebook cell, and add the layers with sub-layers in a separate cell.

martinRenou avatar Sep 15 '22 09:09 martinRenou

I suspect there is a bug when you create the map in a first Notebook cell, and add the layers with sub-layers in a separate cell.

Indeed there is an issue when the Map is shown before adding the layers with sub-items, as seen in this screenshot (the first map view does not have the subitems visible, the second map view has them):

Screenshot from 2022-09-15 11-48-07

martinRenou avatar Sep 15 '22 09:09 martinRenou

please update galata references

HaudinFlorence avatar Sep 21 '22 16:09 HaudinFlorence

please update galata references

HaudinFlorence avatar Sep 22 '22 10:09 HaudinFlorence

please update galata references

HaudinFlorence avatar Sep 22 '22 17:09 HaudinFlorence

please update galata references

martinRenou avatar Sep 23 '22 07:09 martinRenou

please update galata references

martinRenou avatar Sep 23 '22 08:09 martinRenou

Please update galata references, this time it should probably work :D

martinRenou avatar Sep 23 '22 08:09 martinRenou

update galata references

martinRenou avatar Sep 23 '22 09:09 martinRenou

Triggering CI

martinRenou avatar Sep 23 '22 09:09 martinRenou

Thank you!!!

Thanks a lot for the review and help on this PR !

HaudinFlorence avatar Sep 23 '22 09:09 HaudinFlorence