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

Add visibility property in v7 #946

Merged
merged 1 commit into from
Jan 23, 2015
Merged

Add visibility property in v7 #946

merged 1 commit into from
Jan 23, 2015

Conversation

lbud
Copy link
Contributor

@lbud lbud commented Jan 22, 2015

Per mapbox/mapbox-gl-style-spec#212, adds a visibility layout property:
visibility: [visible, placement, none]

placement prevents a layer from rendering but affects other layers' placement; none suppresses a layer altogether.

@jfirebaugh jfirebaugh mentioned this pull request Jan 22, 2015
Merged
@jfirebaugh
Copy link
Contributor

Let's keep placement in our back pocket but use only visible and none for now.

@lbud
Copy link
Contributor Author

lbud commented Jan 22, 2015

Cool, updated.

@jfirebaugh
Copy link
Contributor

👍 Please squash and merge.

@jfirebaugh jfirebaugh modified the milestone: v0.6.0 Jan 22, 2015
lbud pushed a commit that referenced this pull request Jan 23, 2015
Merge pull request #946 from mapbox/visibility
c25473c 
Add visibility property in v7
@lbud lbud merged commit c25473c into master Jan 23, 2015
@lbud lbud deleted the visibility branch January 23, 2015 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@lbud @jfirebaugh