This PR fixes a bug noticed by @kelsey-taylor where a combination of color and opacity expressions in which one used a step expression wasn't handled appropriately. We weren't handling step expressions at all which should have resulted in a black line, but the code was written in a way that allowed a step expression to get somewhere that it shouldn't causing the app to crash rather than just show an inaccurate visual.
For now, this PR adds handling for zoom step expressions only. There's a followup issue to be made for handling scale expressions (step and interpolate) for data expressions in our expandLayers logic. In the meantime, this will warn on those unhandled expressions.
This PR fixes a bug noticed by @kelsey-taylor where a combination of color and opacity expressions in which one used a
stepexpression wasn't handled appropriately. We weren't handlingstepexpressions at all which should have resulted in a black line, but the code was written in a way that allowed astepexpression to get somewhere that it shouldn't causing the app to crash rather than just show an inaccurate visual.For now, this PR adds handling for zoom
stepexpressions only. There's a followup issue to be made for handling scale expressions (stepandinterpolate) for data expressions in ourexpandLayerslogic. In the meantime, this will warn on those unhandled expressions.