It appears this slipped through the code review
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
@Nathan Letwory (jesterking) not sure what we can do on the code review side of thing to help reduce this issue in the future
Comment Actions
If the idea is to hide the high-res smoke part from the UI for 2.82 still, then I'd stick with doing only the UI part. The RNA bit can be done in master.
I leave the actual review to @Sebastián Barschkis (sebbas) and @Brecht Van Lommel (brecht) .
Comment Actions
I'll leave this one for @Sebastián Barschkis (sebbas) to review as well, if it does nothing anymore than seems like a good idea to remove it.