CesiumGS / cesium

An open-source JavaScript library for world-class 3D globes and maps :earth_americas:
https://cesium.com/cesiumjs/
Apache License 2.0
13.02k stars 3.51k forks source link

Better hints to set `enableDebugWireframe` in tileset constructor #10608

Closed j9liu closed 1 year ago

j9liu commented 2 years ago

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:

j9liu commented 2 years ago

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

Sn00pyW00dst0ck commented 2 years ago

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

j9liu commented 2 years ago

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 commented 1 year ago

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