CartoDB / mobile-carto-libs

Internal dependencies for CARTO Mobile SDK
BSD 3-Clause "New" or "Revised" License
4 stars 5 forks source link

properties issues #16

Closed farfromrefug closed 2 years ago

farfromrefug commented 2 years ago

I ll sum up a few issues i am seing here: *text-min-distance: does not seem to do anything . i have tried with very high values (like with places labels) and nothing changes. Is there a special case? What is the unit for it?

mtehver commented 2 years ago

Ok, I will try to answer them one by one:

farfromrefug commented 2 years ago

@mtehver thanks for your answers i remember now about line-pattern-file. I understand what you say. I am just thinking something is wrong as mapbox manage to render them a bit better. By better i mean the "details" are not as "scaled" as in carto. In my screenshot above you can see that we see like a blurred "circle" when the pattern was actually showing a triangle shape. On Mapbox we still see the shape

Screenshot 2021-08-19 at 09 03 48
farfromrefug commented 2 years ago

@mtehver just tested your latest changes, pretty awesome!!! still a weird behavior with text-wrap-width as you can see in the video it seems to not always wrap at the same "place". I use this:

text-wrap-character: '-';
text-wrap-width:step([view::zoom], (9, 100), (15, 180));
text-wrap-before:true;

https://user-images.githubusercontent.com/655344/130239718-5e082e5d-297e-40f5-9c39-b6d334491092.mp4

EDIT: i know why! always thought wrap width was in chars length, it is in pixels! makes sense now

farfromrefug commented 2 years ago

Time to show the progress. It is going really well in copying Mapbox style. Screenshot_1629473300 Screenshot_1629473327

farfromrefug commented 2 years ago

And this is my osm variant. Just a variant of the same style. Screenshot_1629473454

Screenshot_1629473438

mtehver commented 2 years ago

Your styles look great! Any chance of making them publicly available in github?

farfromrefug commented 2 years ago

@mtehver yes i was actually hoping you would ask for this :D Let me clean it up a bit and i ll create a repo just for it.

farfromrefug commented 2 years ago

@mtehver here you go https://github.com/farfromrefug/new_carto_theme

farfromrefug commented 2 years ago

@mtehver it seems there is an issue with line-offset. I dont see it with 'cliffs' when i use it but with this code it draws the "offset" line twice, one on each side

Screenshot 2021-08-21 at 21 07 24

Another side effect of line-offset, on certain zooms i see unwanted lines (same code)

Screenshot 2021-08-21 at 21 11 41
mtehver commented 2 years ago

@farfromrefug What type of geometry do you use with offsetted lines? In case of polygons, this could happen at tile borders if you offset lines 'inwards'. The reason is that polygons are clipped at tile boundaries (with a small extra buffer usually) and when you shift the lines inwards, this clipping becomes visible. Thus this is an artifact of tiling and can not be really fixed (other than increasing the buffer size).

farfromrefug commented 2 years ago

@mtehver it makes sense and yes those are polygons (parks). I ll see what i can do on my side