newSketch versus newSketchOnPlane documentation

marena_richardson

The variable strangeness I was talking about earlier was in newSketch and newSketchOnPlane. Both take a variable called sketchPlane; only newSketchOnPlane says it does in the documentation right now, but they both do according to the starter code in the dropdown menu. However, as far as I can tell newSketch expects a query and newSketchOnPlane expects a plane. 

kevin_o_toole_1

You're right that this is more confusing then it needs to be, but your conclusion is correct: newSketch expects a query for a single plane, while newSketchOnPlane expects a variable with type Plane.

Since this is used so often, we're considering just merging these into one function, such that newSketch(context, id, {sketchPlane : p}); will do the right thing whether p is a plane or a query for a plane. 

Your thoughts on that?

marena_richardson

I think that would be a great idea!