proper fix for top/right axes

Author: nicolaskruchten

src/plots/cartesian/axes.js

@@ -2107,7 +2107,10 @@ axes.doTicks = function(gd, axid, skipTitle) {
 
             if(!ax._marginPush || ax._marginPush < marginPush) {
                 ax._marginPush = marginPush;
-                var pushParams = {x: 0, y: 0, r: 0, l: 0, t: 0, b: 0};
+                var pushParams = {
+                    x: ax.side[0] === 'r' ? 1 : 0,
+                    y: ax.side[0] === 't' ? 1 : 0,
+                    r: 0, l: 0, t: 0, b: 0};
                 pushParams[ax.side[0]] = marginPush;
                 Plots.autoMargin(gd, ax._name, pushParams);
             }

test/image/mocks/long_axis_labels.json

@@ -6,7 +6,7 @@
             "short label 1", "loooooong label 1",
             "short label 2", "loooooong label 2",
             "short label 3", "loooooong label 3",
-            "short label 4", "loooooong label 4",
+            "short label 4", "loooooongloooooongloooooong label 4",
             "short label 5", "loooooong label 5"
         ],
         "y": [
@@ -22,7 +22,7 @@
         "type": "scatter",
         "name": "loooooong y",
         "x":["looooooooooooonger"],
-        "y":["loooooonger"]
+        "y":["loooooooo"]
     }
   ],
     "layout": {
@@ -31,7 +31,7 @@
       "yaxis2": {
         "title": "Y2 Axis Title",
         "overlaying": "y",
-        "side": "right"
+        "side": "right", "titlefont": {"size": 40}
       }
       }
 }