Made the czm_imagerySplitPosition uniform be in pixel coordinates ins…

…tead of relative. Fixes #5106
CesiumGS · Mar 27, 2017 · 20a54f3 · 20a54f3
1 parent 201e3f7
commit 20a54f3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/Source/Renderer/AutomaticUniforms.js b/Source/Renderer/AutomaticUniforms.js
@@ -1469,7 +1469,7 @@ define([
 
          /**
          * An automatic GLSL uniform representing the splitter position to use when rendering imagery layers with a splitter.
-         * This will be in the range 0.0 to 1.0 with 0.0 being the far left of the viewport and 1.0 being the far right of the viewport.
+         * This will be in pixel coordinates relative to the canvas.
          *
          * @alias czm_imagerySplitPosition
          * @glslUniform

diff --git a/Source/Renderer/UniformState.js b/Source/Renderer/UniformState.js
@@ -951,7 +951,8 @@ define([
         this._frameState = frameState;
         this._temeToPseudoFixed = Transforms.computeTemeToPseudoFixedMatrix(frameState.time, this._temeToPseudoFixed);
 
-        this._imagerySplitPosition = frameState.imagerySplitPosition;
+        // Convert the relative imagerySplitPosition to absolute pixel coordinates
+        this._imagerySplitPosition = frameState.imagerySplitPosition * canvas.clientWidth;
         var fov = camera.frustum.fov;
         var viewport = this._viewport;
         var pixelSizePerMeter;

diff --git a/Source/Shaders/GlobeFS.glsl b/Source/Shaders/GlobeFS.glsl
@@ -99,7 +99,7 @@ vec4 sampleAndBlend(
     float alpha = value.a;
 
 #ifdef APPLY_SPLIT
-    float splitPosition = czm_imagerySplitPosition * czm_viewport.z;
+    float splitPosition = czm_imagerySplitPosition;
     // Split to the left
     if (split < 0.0 && gl_FragCoord.x > splitPosition) {
        alpha = 0.0;