Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Circles without correspnding label appearing #167

Closed
daniel-j-h opened this issue Oct 28, 2023 · 3 comments
Closed

Circles without correspnding label appearing #167

daniel-j-h opened this issue Oct 28, 2023 · 3 comments

Comments

@daniel-j-h
Copy link

When zooming into the map on some zoom layers it looks like circle appear without corresponding labels.

Have a look at the screenshots below; in the second one I'm highlighting the issue in red (zoom in).

I believe this is the case because we have

  • a circle layer
  • a symbol layer for the text, with an anchor on the left (or center, conditionally)

I think this is how we could fix it

  1. We don't do anything and don't care that circles without labels are appearing
  2. Instead of a circle layer and a symbol layer for text only, we could have a single symbol layer and use text+icon in a single layer
  3. We add variable anchors so that the labels still show up when the left/center anchor doesn't work out

https://maplibre.org/maplibre-style-spec/layers/#layout-symbol-text-variable-anchor

Note: the screenshots are from the latest map on https://maps.protomaps.com

basemap0
basemap1
basemap2

@nvkelso
Copy link
Collaborator

nvkelso commented Oct 28, 2023 via email

@bdon
Copy link
Member

bdon commented Oct 28, 2023

This is related to #160 because to make text + circle into a unified layer we need to use the Symbol layer type.

I just created https://github.com/protomaps/basemaps-assets with placeholder assets - I think using GitHub pages with no build step is the most obvious and transparent place to put sprites/fonts for the basemap. will begin to flesh this out and gather some feedback before we commit to using those URLs

bdon added a commit that referenced this issue May 8, 2024
bdon added a commit that referenced this issue May 8, 2024
* Initial sprite generation code. [#238, #167]

* update townspot themes [#167]

* support 1x and 2x ratios

* pin pmtiles dependency; fix style diffing when changing themes
bdon added a commit that referenced this issue May 9, 2024
* Townspots are rendered as part of the symbol layer and require a theme-specific spritesheet.
* Deployments of basemaps style v3.0.0 requires specifying a sprite MapLibre style property just like fontstacks.
bdon added a commit that referenced this issue May 9, 2024
* Townspots are rendered as part of the symbol layer and require a theme-specific spritesheet.
* Deployments of basemaps style v3.0.0 requires specifying a sprite MapLibre style property just like fontstacks.
bdon added a commit that referenced this issue May 10, 2024
* Townspots are rendered as part of the symbol layer and require a theme-specific spritesheet.
* Deployments of basemaps style v3.0.0 requires specifying a sprite MapLibre style property just like fontstacks.
bdon added a commit that referenced this issue May 10, 2024
* Townspots are rendered as part of the symbol layer and require a theme-specific spritesheet.
* Deployments of basemaps style v3.0.0 requires specifying a sprite MapLibre style property just like fontstacks.
bdon added a commit that referenced this issue May 10, 2024
* remove unused city_circle and city_circle_stroke theme variables. [#238]

* styles v3.0.0: Introduce spritesheets [#238, #167]

* Townspots are rendered as part of the symbol layer and require a theme-specific spritesheet.
* Deployments of basemaps style v3.0.0 requires specifying a sprite MapLibre style property just like fontstacks.

* update changelog copy
@bdon
Copy link
Member

bdon commented May 10, 2024

This is resolved with the addition of spritesheets in styles v3.0.0

Screenshot 2024-05-10 at 16 18 30

@bdon bdon closed this as completed May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants