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

Outline breaks with perPositionHeight and extrudedHeight together #6595

Closed
Vineg opened this issue May 17, 2018 · 4 comments · Fixed by #6602
Closed

Outline breaks with perPositionHeight and extrudedHeight together #6595

Vineg opened this issue May 17, 2018 · 4 comments · Fixed by #6602

Comments

@Vineg
Copy link
Contributor

Vineg commented May 17, 2018

var viewer = new Cesium.Viewer('cesiumContainer');

var orangePolygon = viewer.entities.add({
    name : 'Orange polygon with per-position heights and outline',
    polygon : {
        hierarchy : Cesium.Cartesian3.fromDegreesArrayHeights([-108.0, 25.0, 100000,
                                                               -100.0, 25.0, 100000,
                                                               -100.0, 30.0, 100000,
                                                               -108.0, 30.0, 300000]),
        perPositionHeight : true,
        material : Cesium.Color.ORANGE.withAlpha(0.5),
        outline : true,
        outlineColor : Cesium.Color.BLACK,
        extrudedHeight : 100000
    }
});

viewer.zoomTo(viewer.entities);

image

@Vineg Vineg changed the title Outline breaks when dynamically updating extrudedHeight Outline breaks with perPositionHeight and extrudedHeight together May 17, 2018
@Vineg
Copy link
Contributor Author

Vineg commented May 17, 2018

@hpinkos maybe it is related to this commit cd60c47 ?

@hpinkos
Copy link
Contributor

hpinkos commented May 21, 2018

Thanks @Vineg! Do you know if this worked in a previous version of Cesium?

@Vineg
Copy link
Contributor Author

Vineg commented May 21, 2018

@hpinkos Yes, it was fine in 41 version

@hpinkos
Copy link
Contributor

hpinkos commented May 21, 2018

Thanks! I'll try to look at before the next release

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

Successfully merging a pull request may close this issue.

2 participants