Skip to content
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

ie11 problem #4091

Closed
lucasvw opened this issue Jul 6, 2016 · 3 comments
Closed

ie11 problem #4091

lucasvw opened this issue Jul 6, 2016 · 3 comments

Comments

@lucasvw
Copy link

lucasvw commented Jul 6, 2016

Hi guys,

We just encountered an ie11 problem. Cesium throws an error upon using

viewer.scene.pickPosition()

TypeError: Object doesn't support property or method 'fill'
TypeError: Object doesn't support property or method 'fill'
at T (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:459:30490)
at S.prototype.setColor (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:460:1172)
at color.set (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:459:28961)
at Anonymous function (http://localhost/vcm/src/vcs/vcm/layer/cesium/buildingsLayer.js:185:25)
at Anonymous function (http://localhost/vcm/src/vcs/vcm/layer/cesium/buildingsLayer.js:168:21)
at vcs.vcm.layer.cesium.Buildings.prototype.applyStyle (http://localhost/vcm/src/vcs/vcm/layer/cesium/buildingsLayer.js:157:13)
at i.prototype.raiseEvent (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:423:24310)
at U (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:463:13524)
at T.prototype.update (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:463:16244)
at a.prototype.update (http://localhost/vcm/lib/thirdparty/Cesium/Cesium.js:470:4541)

We could trace it back to the following commit: 24c199c

Would you consider reverting this commit?

Best,
Lucas

@mramato
Copy link
Contributor

mramato commented Jul 6, 2016

Thanks @lucasvw, this is a known issue that is discussed in #3614. The 3D-Tiles branch is in development and this will get fixed before it is officially released (but I don't have a specific time frame for you yet).

@lilleyse perhaps someone can look at this soon?

@mramato mramato closed this as completed Jul 6, 2016
@lucasvw
Copy link
Author

lucasvw commented Jul 6, 2016

wow, that was a fast reply :)
Thanks!

@lilleyse
Copy link
Contributor

lilleyse commented Jul 6, 2016

@lilleyse perhaps someone can look at this soon?

Yup I'll take a look soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants