Setting wireframe rendering on material instead of geometry

[simonharrisco]

Currently you set the primitive when you define the geometry
eg:

renderer.geometry({
  GeometryComponent,
  primitive: graph.ctx.Primitive.Lines
})

Is this the best place to declare this? Would it made more sense to have it set within the material?
Is this a more logical place to put it as materials effect how things look?

Threejs does this

material = new THREE.MeshBasicMaterial({
    color: 0xff0000,
    wireframe: true
});

BabylonJs does this

var myMaterial = new BABYLON.StandardMaterial("myMaterial", scene);
myMaterial.wireframe = true;

This reasoning breaks down if the primitive type given in the geometry also supports things like TRIANGLE_STRIP,TRIANGLE FAN ETC.

What does everyone think?

[dmnsgn]

To sum up, should the drawing primitive be tied to geometry or material. Wireframe is a bit different than just setting the primitive to LINES. Three.js has the primitive on the mesh level: https://threejs.org/docs/#api/en/constants/DrawModes

It does affect how things look and is related to a program so I tend to agree with moving it in the Material which is already handling blend, depth, culling...

[dmnsgn]

v4: better off using a material type while we see if gl.Lines needs to be abstracted in its own renderer #311 (comment)