cesium icon indicating copy to clipboard operation
cesium copied to clipboard

Better hints to set `enableDebugWireframe` in tileset constructor

Open j9liu opened this issue 3 years ago • 3 comments

As of Cesium 1.97, the enableDebugWireframe flag will be required for debugWireframe to work, for both Model and Cesium3DTileset. Otherwise, nothing happens when debugWireframe is toggled. We should more clearly convey to the user that enableDebugWireframe needs to be set true for debugWireframe to work.

Some suggestions:

  • oneTimeWarning if a user is setting debugWireframe: true in the tileset constructor without setting enableDebugWireframe true, or if a user is setting debugWireframe at runtime
  • same for Model
  • disabling the wireframe checkbox in Cesium3DTilesInspector, with some text that enableDebugWireframe should be true for this to work`

j9liu avatar Jul 28 '22 16:07 j9liu

Link to the forum post that brought this up: https://community.cesium.com/t/debugwireframe-not-working-with-3dtiles-content-gltf/19741/4

j9liu avatar Aug 01 '22 15:08 j9liu

I can take a look into implementing this if nobody else has started working on it.

Sn00pyW00dst0ck avatar Sep 13 '22 00:09 Sn00pyW00dst0ck

Thanks for your interest in contributing @Sn00pyW00dst0ck! I don't believe anyone has started working on it, so feel free to contribute and open a PR when you're able 😀

j9liu avatar Sep 14 '22 17:09 j9liu

Looks like this was addressed by #10803, so I'm closing this issue.

j9liu avatar Dec 14 '22 21:12 j9liu