From 2d7fa9027bee49bbbca0943ad708680477c47809 Mon Sep 17 00:00:00 2001
From: Anthony Pessy <anthony@cogniteev.com>
Date: Sat, 14 Apr 2018 09:42:23 +0200
Subject: [PATCH] The zoom() getter now correctly returns current domain

This fix is copied from #2270 by @robhybrid.

Closes #1497
---
 spec/api.zoom-spec.js | 8 ++++++++
 src/api.zoom.js       | 4 +++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/spec/api.zoom-spec.js b/spec/api.zoom-spec.js
index adba585..8ea728e 100644
--- a/spec/api.zoom-spec.js
+++ b/spec/api.zoom-spec.js
@@ -24,6 +24,14 @@ describe('c3 api zoom', function () {
             };
         });
 
+        it('should return the correct extent', function () {
+            var zoomDomain = chart.zoom(),
+                expectedDomain = chart.internal.x.domain();
+
+            expect(+zoomDomain[0]).toBe(+expectedDomain[0]);
+            expect(+zoomDomain[1]).toBe(+expectedDomain[1]);
+        });
+
         it('should be zoomed properly', function () {
             var target = [3, 5], domain;
             chart.zoom(target);
diff --git a/src/api.zoom.js b/src/api.zoom.js
index 863bcbe..2f4ba09 100644
--- a/src/api.zoom.js
+++ b/src/api.zoom.js
@@ -15,8 +15,10 @@ c3_chart_fn.zoom = function (domain) {
             $$.redraw({withY: $$.config.zoom_rescale, withSubchart: false});
         }
         $$.config.zoom_onzoom.call(this, $$.x.orgDomain());
+        return domain;
+    } else {
+        return $$.x.domain();
     }
-    return domain;
 };
 c3_chart_fn.zoom.enable = function (enabled) {
     var $$ = this.internal;