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

Do non-name fields work for text? #394

Closed
yhahn opened this issue May 29, 2014 · 5 comments
Closed

Do non-name fields work for text? #394

yhahn opened this issue May 29, 2014 · 5 comments

Comments

@yhahn
Copy link
Member

yhahn commented May 29, 2014

I feel like I haven't seen any code that would prevent this but just needs a quick confirm. Native counterpart: mapbox/mapbox-gl-native#272

cc @mikemorris
@mikemorris
Copy link
Contributor

Only feature.name is currently being shaped https://github.com/mapbox/llmr/blob/master/js/worker/workertile.js#L178

@yhahn
Copy link
Member Author

yhahn commented May 29, 2014

sadtrombone

@ansis
Copy link
Contributor

ansis commented May 29, 2014

9d716c3
fr

@ansis ansis closed this as completed May 29, 2014
@springmeyer
Copy link
Contributor

Danemark 👍

@mikemorris
Copy link
Contributor

Awesome @ansis!

bensleveritt pushed a commit to bensleveritt/mapbox-gl-js that referenced this issue Oct 24, 2016
@mcwhittemore
move getFeaturesAt to use point. Closes mapbox#394
f57fe93
lucaswoj pushed a commit that referenced this issue Jan 11, 2017
@samanpwbb
require ids on layers, closes #394
ea9b3ef
lucaswoj pushed a commit that referenced this issue Jan 11, 2017
@samanpwbb
Merge pull request #395 from mapbox/require-layer-ids
d023533 
require ids on layers, closes #394
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@springmeyer @yhahn @mikemorris @ansis