Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Partial implementation of runtime styling API for node bindings #5318

Merged
merged 6 commits into from
Jun 14, 2016

Conversation

jfirebaugh
Copy link
Contributor

Enough to pass the three basic runtime styling tests in test-suite. 🎉

Still TODO:

  • setLayoutProperty
  • setPaintProperty with a function
  • Support color, translate, dash-array, and font-stack properties

@jfirebaugh
Copy link
Contributor Author

There will be some conflicts with #5288; I'm going to wait for the merge there and then rebase and resolve them here.

@jfirebaugh jfirebaugh force-pushed the node-style-api branch 4 times, most recently from 30948de to 1b4b3b1 Compare June 13, 2016 23:52
Rewrite enum.hpp in such a way that parseConstant can be defined generically for all enumerated types.

While there, properly validated enumerated property values.
For runtime styling, it needs to overwrite any existing value.
@jfirebaugh jfirebaugh closed this Jun 14, 2016
@jfirebaugh jfirebaugh deleted the node-style-api branch June 14, 2016 00:46
@jfirebaugh jfirebaugh merged commit 3f5ea16 into master Jun 14, 2016
@mikemorris mikemorris added the Node.js node-mapbox-gl-native label Jul 14, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Node.js node-mapbox-gl-native
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants