Some Suggestions/Wish List

Jan 26, 2012 at 6:32 PM

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. 
Jan 26, 2012 at 7:09 PM

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

Jan 26, 2012 at 7:13 PM

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

Jan 26, 2012 at 8:13 PM

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