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.

Sign up for GitHub

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

Jump to bottom

support fontsize as function for placement #317

Closed
ansis opened this issue Mar 1, 2014 · 4 comments
Closed

support fontsize as function for placement #317

ansis opened this issue Mar 1, 2014 · 4 comments
Labels
feature 🍏

Comments

@ansis
Copy link
Contributor

ansis commented Mar 1, 2014

If the fontsize increases a lot as you zoom in, then just using the largest fontsize during placement isn't good enough. It adds too much padding around labels and reduces density.

placement with "fontSize": 40

placement with "fontSize": 20
@ansis ansis mentioned this issue Mar 1, 2014
Closed
@mourner
Copy link
Member

mourner commented Mar 1, 2014

@ansis why does the text look pretty much the same size while having the font size value halved?

@mourner
Copy link
Member

mourner commented Mar 1, 2014

Oh, I get it now — placement is done once for a round zoom and only considers max font size in the scale range.

@mourner
Copy link
Member

mourner commented Mar 15, 2014

Pretty high priority now it seems, although I'm not sure how to approach this. @ansis thoughts?

@mourner mourner mentioned this issue Apr 9, 2014
Closed
@mourner mourner added this to the future milestone Jun 24, 2014
@mourner
Copy link
Member

mourner commented Jan 29, 2015

Done with #361

@mourner mourner closed this as completed Jan 29, 2015
bensleveritt pushed a commit to bensleveritt/mapbox-gl-js that referenced this issue Oct 24, 2016
@mcwhittemore
Merge pull request mapbox#317 from mapbox/multi____
de42099 
Adds MultiPolygon, MultiLineString and MultiPoint supprt
lucaswoj pushed a commit that referenced this issue Jan 11, 2017
@jfirebaugh
Merge pull request #317 from mapbox/pbf_instead_png
2479011 
Fix misleading source example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature 🍏
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mourner @ansis