przem321 wrote at 2012-01-26 19:32:
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;
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.
Customer support service by UserEcho