ignore drawAxis on y2 if there’s no y2 axis

fixes tests/unboxed-spark
danvk · Feb 16, 2023 · 046bdec · 046bdec
1 parent 1655f36
commit 046bdec
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGES.md b/CHANGES.md
@@ -15,6 +15,7 @@
 - Allow ES6-importing the extras (#848) (#989) (#1009)
 - Fix broken zoom also in the drawing test (same as #611/#953)
 - Resetting `panEdgeFraction` now works
+- Fix tests/unboxed-spark regression caused by PR #936
 
 ## Other user-visible changes
 - Fix issue #611 in the drawing demo gallery (#953)

diff --git a/src/plugins/axes.js b/src/plugins/axes.js
@@ -162,7 +162,8 @@ axes.prototype.willDrawChart = function(e) {
 
   const that = this;
 
-  if (g.getOptionForAxis('drawAxis', 'y') || g.getOptionForAxis('drawAxis', 'y2')) {
+  if (g.getOptionForAxis('drawAxis', 'y') ||
+      (g.numAxes() == 2 && g.getOptionForAxis('drawAxis', 'y2'))) {
     if (layout.yticks && layout.yticks.length > 0) {
       var num_axes = g.numAxes();
       var getOptions = [makeOptionGetter('y'), makeOptionGetter('y2')];