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

Primitive ordering regression #6569

Open
hpinkos opened this issue May 7, 2018 · 2 comments
Open

Primitive ordering regression #6569

hpinkos opened this issue May 7, 2018 · 2 comments

Comments

@hpinkos
Copy link
Contributor

hpinkos commented May 7, 2018

In Cesium 1.29, the using raiseToTop makes the polyline render on top of the billboard and label. However, in Cesium 1.30 this is no longer the case.

Reported by @erezy

https://groups.google.com/forum/#!topic/cesium-dev/ywyFR5GSi_c

var viewer = new Cesium.Viewer('cesiumContainer',{
    sceneMode: Cesium.SceneMode.SCENE2D 
});

var polylines = new Cesium.PolylineCollection();
polylines.add({
  positions : Cesium.Cartesian3.fromDegreesArray([
    -75.10, 39.57,
    -77.02, 38.53,
    -80.50, 35.14,
    -80.12, 25.46]),
  width : 2,
   material : new Cesium.Material({
    fabric : {
        type : 'Color',
        uniforms : {
            color : new Cesium.Color(1.0, 1.0, 0.0, 1.0)
        }
    }
})
});

var billboards = new Cesium.BillboardCollection();
billboards.add({
  position : Cesium.Cartesian3.fromDegrees(-77.02, 38.53),
  image : '../images/Cesium_Logo_overlay.png'
});
var labels = new Cesium.LabelCollection();
labels.add({
  position : Cesium.Cartesian3.fromDegrees(-80.50, 35.14),
    text : 'A label',
    fillColor: Cesium.Color.BLUE
});

var primitives = viewer.scene.primitives;
primitives.add(labels);
primitives.add(billboards);  // Add collection
primitives.add(polylines);

primitives.raiseToTop(polylines);
@erezy
Copy link

erezy commented Nov 14, 2018

Hi @hpinkos ,
do you have news on this issue? are you still working on it?

@hpinkos
Copy link
Contributor Author

hpinkos commented Nov 15, 2018

@erezy no sorry, we haven't had a chance to look into this. I'm guessing the regression came with this PR: #4886

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

No branches or pull requests

2 participants