revert bringPortalsToFront change - and add some comments concerning possible better fixes
This commit is contained in:
parent
c74971e84e
commit
3c6468cccb
@ -114,8 +114,15 @@ window.Render.prototype.processGameEntities = function(entities) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// reorder portals to be after links/fields
|
//TODO: better method to bring portals to front during rendering
|
||||||
this.bringPortalsToFront();
|
//as it stands, rendering from cache causes multiple passes through bringToFront - which is a waste
|
||||||
|
//possible options:
|
||||||
|
//1. add fields, links and portals in the order they should be rendered. will be close to right while rendering, and
|
||||||
|
// a final bringPortalsToFront call can fix up any errors
|
||||||
|
//2. run bringPortalsToFront on a short timer, so it's not immediate after render.
|
||||||
|
|
||||||
|
// // reorder portals to be after links/fields
|
||||||
|
// this.bringPortalsToFront();
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -143,8 +150,8 @@ window.Render.prototype.endRenderPass = function() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// // reorder portals to be after links/fields
|
// reorder portals to be after links/fields
|
||||||
// this.bringPortalsToFront();
|
this.bringPortalsToFront();
|
||||||
|
|
||||||
this.isRendering = false;
|
this.isRendering = false;
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user