-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add clearer error message #6405
Conversation
@ggetz, thanks for the pull request! Maintainers, we have a signed CLA from @ggetz, so you can review this at any time.
I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
Source/Scene/DracoLoader.js
Outdated
@@ -177,6 +177,12 @@ define([ | |||
return when.resolve(); | |||
} | |||
|
|||
if (FeatureDetection.isInternetExplorer()) { | |||
return when.reject({ | |||
message : 'Draco decoding is not supported in legacy environments.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be a little more obvious, maybe just say "Draco decoding is not currently support in Internet Explorer"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True, I was just using the terminology the module itself reports.
@ggetz Add a comment about this in |
Thanks @hpinkos, updated! |
One more thing (sorry for not thinking of this before) |
And probably also mention the |
|
Oooh, gotcha. Yeah, nevermind then, I think this is fine =) |
Thanks @ggetz! |
Fixes ##6399 with a clearer error.
Opened #6404 for a better fix by using the legacy enabled build of the draco decoder module.