Temp fix for yref=paper

hannahker · hannahker · commit 2b59e2c7a436 · 2023-02-06T18:03:29.000-08:00
diff --git a/src/plot_api/subroutines.js b/src/plot_api/subroutines.js
@@ -407,8 +407,6 @@ exports.drawMainTitle = function(gd) {
 
     if(gd._fullLayout.title.automargin) {
         applyTitleAutoMargin(gd, fullLayout, dy);
-        // A bit hacky to make sure top padding is properly accounted for
-        y += gd._fullLayout.title.pad.t;
     }
 
     Titles.draw(gd, 'gtitle', {
@@ -447,7 +445,7 @@ function applyTitleAutoMargin(gd, fullLayout, dy) {
     // TODO: Push titleDepth depending on yanchor?
     push[direction] = (
         titleDepth(title) +
-        getMainTitleY(fullLayout, dy) +
+        // getMainTitleY(fullLayout, dy) + // TODO: Why is this calculation triggering too many automargin redraws?
         title.pad.t +
         title.pad.b
         );
diff --git a/test/image/mocks/zzz-automargin-title.json b/test/image/mocks/zzz-automargin-title.json
@@ -25,10 +25,12 @@
         },
         "title": {
             "automargin": true, 
-            "text": "Some text",
-            "y": 1,
-            "yanchor": "top",
-            "container": "paper"
+            "text": "A sample title",
+            "font": {"size": 24},
+            "pad": {"t": 15},
+            "yanchor": "bottom",
+            "yref": "paper",
+            "y": 1
         }
   }
 }
