sohetma
diff --git a/‎src/plots/cartesian/layout_defaults.js
Lines changed: 5 additions & 0 deletions b/‎src/plots/cartesian/layout_defaults.js
Lines changed: 5 additions & 0 deletions
diff --git a/‎src/traces/image/calc.js
Lines changed: 1 addition & 1 deletion b/‎src/traces/image/calc.js
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/traces/image/hover.js
Lines changed: 2 additions & 2 deletions b/‎src/traces/image/hover.js
Lines changed: 2 additions & 2 deletions
diff --git a/‎src/traces/image/plot.js
Lines changed: 2 additions & 2 deletions b/‎src/traces/image/plot.js
Lines changed: 2 additions & 2 deletions
diff --git a/‎test/image/baselines/image_axis_reverse.png
-1.58 KB b/‎test/image/baselines/image_axis_reverse.png
-1.58 KB
diff --git a/‎test/image/baselines/image_cat.png
-277 Bytes b/‎test/image/baselines/image_cat.png
-277 Bytes
diff --git a/‎test/image/mocks/image_adventurer.json
Lines changed: 1 addition & 1 deletion b/‎test/image/mocks/image_adventurer.json
Lines changed: 1 addition & 1 deletion
diff --git a/‎test/image/mocks/image_axis_reverse.json
Lines changed: 2 additions & 2 deletions b/‎test/image/mocks/image_axis_reverse.json
Lines changed: 2 additions & 2 deletions
diff --git a/‎test/image/mocks/image_cat.json
Lines changed: 1 addition & 1 deletion b/‎test/image/mocks/image_cat.json
Lines changed: 1 addition & 1 deletion
@@ -87,6 +87,11 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
             if(!traceIs(trace, 'carpet') || (trace.type === 'carpet' && !trace._cheater)) {
                 if(xaName) xaMustDisplay[xaName] = true;
             }
+
+            if(trace.type === 'image') {
+                if(yaName) yaMustForward[yaName] = false;
+                if(yaName) yaMayBackward[yaName] = true;
+            }
         }
 
         // Two things trigger axis visibility:
 
@@ -22,7 +22,7 @@ module.exports = function calc(gd, trace) {
     var w = trace.z[0].length;
 
     trace._extremes[xa._id] = Axes.findExtremes(xa, [x0, x0 + w * trace.dx]);
-    trace._extremes[ya._id] = Axes.findExtremes(ya, [y0, y0 - h * trace.dy]);
+    trace._extremes[ya._id] = Axes.findExtremes(ya, [y0, y0 + h * trace.dy]);
 
     var cd0 = {
         x0: x0,
 
@@ -19,15 +19,15 @@ module.exports = function hoverPoints(pointData, xval, yval) {
     var ya = pointData.ya;
 
     if(Fx.inbox(xval - cd0.x0, xval - (cd0.x0 + cd0.w * trace.dx), 0) > 0 ||
-            Fx.inbox(yval - cd0.y0, yval - (cd0.y0 - cd0.h * trace.dy), 0) > 0) {
+            Fx.inbox(yval - cd0.y0, yval - (cd0.y0 + cd0.h * trace.dy), 0) > 0) {
         return;
     }
 
     // Find nearest pixel's index and pixel center
     var nx = Math.floor((xval - cd0.x0) / trace.dx);
     var ny = Math.floor(Math.abs(yval - cd0.y0) / trace.dy);
 
-    var py = ya.c2p(cd0.y0 - (ny + 0.5) * trace.dy);
+    var py = ya.c2p(cd0.y0 + (ny + 0.5) * trace.dy);
     return [Lib.extendFlat(pointData, {
         index: [ny, nx],
         x0: xa.c2p(cd0.x0 + nx * trace.dx),
 
@@ -31,7 +31,7 @@ module.exports = function(gd, plotinfo, cdimage, imageLayer) {
         var left = xa.c2p(x0);
         var right = xa.c2p(x0 + w * dx);
         var top = ya.c2p(y0);
-        var bottom = ya.c2p(y0 - h * dy);
+        var bottom = ya.c2p(y0 + h * dy);
 
         var temp;
         if(right < left) {
@@ -69,7 +69,7 @@ module.exports = function(gd, plotinfo, cdimage, imageLayer) {
         canvas.height = imageHeight;
         var context = canvas.getContext('2d');
         var ipx = function(i) {return Lib.constrain(Math.round(xa.c2p(x0 + i * dx) - left), 0, imageWidth);};
-        var jpx = function(j) {return Lib.constrain(Math.round(ya.c2p(y0 - j * dy) - top), 0, imageHeight);};
+        var jpx = function(j) {return Lib.constrain(Math.round(ya.c2p(y0 + j * dy) - top), 0, imageHeight);};
         // TODO: for performance, when image size is reduced, only loop over pixels of interest
         for(var i = 0; i < cd0.w; i++) {
             for(var j = 0; j < cd0.h; j++) {
 
@@ -13,8 +13,8 @@
     "width": 600, "height": 600,
     "margin": {"t": 35, "l": 35, "b": 35, "r": 35},
     "xaxis2": {"autorange": "reversed"},
-    "yaxis3": {"autorange": "reversed"},
-    "yaxis4": {"autorange": "reversed"},
+    "yaxis3": {"autorange": true},
+    "yaxis4": {"autorange": true},
     "xaxis4": {"autorange": "reversed"}
   }
 }
Original file line number	Diff line number	Diff line change
`@@ -87,6 +87,11 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {`
`87`	`87`	`if(!traceIs(trace, 'carpet') \|\| (trace.type === 'carpet' && !trace._cheater)) {`
`88`	`88`	`if(xaName) xaMustDisplay[xaName] = true;`
`89`	`89`	`}`
	`90`	`+`
	`91`	`+ if(trace.type === 'image') {`
	`92`	`+ if(yaName) yaMustForward[yaName] = false;`
	`93`	`+ if(yaName) yaMayBackward[yaName] = true;`
	`94`	`+ }`
`90`	`95`	`}`
`91`	`96`
`92`	`97`	`// Two things trigger axis visibility:`
Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@`
`13`	`13`	`"width": 600, "height": 600,`
`14`	`14`	`"margin": {"t": 35, "l": 35, "b": 35, "r": 35},`
`15`	`15`	`"xaxis2": {"autorange": "reversed"},`
`16`		`- "yaxis3": {"autorange": "reversed"},`
`17`		`- "yaxis4": {"autorange": "reversed"},`
	`16`	`+ "yaxis3": {"autorange": true},`
	`17`	`+ "yaxis4": {"autorange": true},`
`18`	`18`	`"xaxis4": {"autorange": "reversed"}`
`19`	`19`	`}`
`20`	`20`	`}`