You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Aug 8, 2023. It is now read-only.
Repository navigation
This repository was archived by the owner on Aug 8, 2023. It is now read-only.
Basic runtime styling support was added to the Node module in #5318, but it doesn’t include property functions, which were implemented in mbgl in #7372. Currently, you can set property functions via style JSON. This issue tracks adding a way to manipulate them via the runtime styling API.
@jfirebaugh, I was referring to passing property functions into setPaintProperty() from the JavaScript side, going off the discussion in #7372 (comment). From a cursory look at node_map.cpp, it didn’t look like there was support for property functions, but if there’s conversion magic going on, we can close this ticket and call it done. 😄
Activity
jfirebaugh commentedon Mar 22, 2017
Can you say more specifically what you see as missing? The existing bindings pass most of the tests in https://github.com/mapbox/mapbox-gl-js/tree/master/test/integration/render-tests/runtime-styling, which include tests that use property functions.
1ec5 commentedon Mar 22, 2017
@jfirebaugh, I was referring to passing property functions into
setPaintProperty()from the JavaScript side, going off the discussion in #7372 (comment). From a cursory look at node_map.cpp, it didn’t look like there was support for property functions, but if there’s conversion magic going on, we can close this ticket and call it done. 😄jfirebaugh commentedon Mar 22, 2017
Yep, that should work -- it's handled by the logic in
style/conversion/function.hpp.