search

GeoTIFF / georaster-layer-for-leaflet

Display GeoTIFFs and soon other types of raster on your Leaflet Map
https://geotiff.github.io/georaster-layer-for-leaflet-example/
Apache License 2.0
297 stars 58 forks source link

Unstable drawing 3.0.1 #74

Closed ayerix closed 3 years ago

ayerix commented 3 years ago

Geotiff attached below works stable on 2.0.2 and has significant gaps (different on different scales) on 3.0.1 See geotiff and drawing examples in attach __Chel4_v2.zip screens.zip leaflet 1.7.1, georaster parser 1.5.4

DanielJDufour commented 3 years ago

Hi, @ayerix . Thanks for sharing this. I'm sorry for the inconvenience this has caused. Does it load correctly when you load the GeoTIFF at https://geotiff.github.io/georaster-layer-for-leaflet-example/examples/load-file.html?

ayerix commented 3 years ago

Works ok here, no missing straps, not reproduced. But I don't see plugin versions, so I can't say for sure about it's difference

DanielJDufour commented 3 years ago

I updated the example to use Leaflet 1.7.1 and it appears to continue to work. Here's the new versions:

Could you try with georaster-layer-for-leaflet 3.1.2 and see if any of the recent changes fixed the bug for you?

Could it be a CSS issue? I'm sorry I haven't been more helpful.

benhid commented 3 years ago

@DanielJDufour CSS issue indeed!

I'm using tailwindcss and it comes with a set of base styles -Preflight- built on top of modern-normalize.

It seems like modern-normalize sets box-sizing: border-box. Hence using

.leaflet-container * {
    box-sizing: initial;
}

solved the issue for me 👏🏻

DanielJDufour commented 3 years ago

hey, @ayerix and @benhid . Thank you for bringing this up and figuring this out! I made a fix and republished the library, so you shouldn't need to use .leaflet-container * { box-sizing: initial; } anymore. You really helped people me make sure GeoRasterLayer works with tailwindcss and other css libraries. Thank you!

Please open any more issues you find.