Some Suggestions/Wish List

przem321 wrote at 2012-01-26 19:32:

Hi Objo,

here are some issues I would like to mention:

• PointsVisual3D will not work if Points is set to null. While this is not a bug, I like to disable the visual without removing it from the Viewport.Childern by setting Poins=null. I know, there is the IsRendering prop, but it serves a different purpose. Would it not make sense to add a line like 

	if (Points == null) return; 

to the 

	protected override void UpdateGeometry() function? 

The same applies to all ScreenSpaceVisual3D classes?

• Shouldn't the MeshNormals3D class be called MeshNormalsVisual3D for sake of consistency? 
• The BoundingBoxVisual3D has a hard-coded color to Yellow - well, not the perfect solution. I added a Property "Fill" to it, there one can set the color. I submit the file as a patch. 

objo wrote at 2012-01-26 20:09:

thanks for reporting bugs/inconsistencies! I'm correcting all three now.

objo wrote at 2012-01-26 20:13:

I used the property name "Color" instead of "Fill" (which should be of type Brush).

objo wrote at 2012-01-26 21:13:

After one more thought I changed again to "Fill" as you suggested, to be consistent with the other objects in the library.