From 89edb98bed6c3b40a8829bcfb6343cd29d132d85 Mon Sep 17 00:00:00 2001
From: alexcjohnson <alex@plot.ly>
Date: Sun, 29 Apr 2018 12:57:05 -0400
Subject: [PATCH] update modebar buttons when locale changes

---
 src/components/modebar/modebar.js   |  9 +++++---
 test/jasmine/tests/localize_test.js | 34 +++++++++++++++++++++++++++++
 2 files changed, 40 insertions(+), 3 deletions(-)

diff --git a/src/components/modebar/modebar.js b/src/components/modebar/modebar.js
index 306e5cdcf9c..1b12ed8bf45 100644
--- a/src/components/modebar/modebar.js
+++ b/src/components/modebar/modebar.js
@@ -52,10 +52,13 @@ proto.update = function(graphInfo, buttons) {
     else this.element.className = 'modebar';
 
     // if buttons or logo have changed, redraw modebar interior
-    var needsNewButtons = !this.hasButtons(buttons),
-        needsNewLogo = (this.hasLogo !== context.displaylogo);
+    var needsNewButtons = !this.hasButtons(buttons);
+    var needsNewLogo = (this.hasLogo !== context.displaylogo);
+    var needsNewLocale = (this.locale !== context.locale);
 
-    if(needsNewButtons || needsNewLogo) {
+    this.locale = context.locale;
+
+    if(needsNewButtons || needsNewLogo || needsNewLocale) {
         this.removeAllButtons();
 
         this.updateButtons(buttons);
diff --git a/test/jasmine/tests/localize_test.js b/test/jasmine/tests/localize_test.js
index ec37e586f75..34758cce2d9 100644
--- a/test/jasmine/tests/localize_test.js
+++ b/test/jasmine/tests/localize_test.js
@@ -369,4 +369,38 @@ describe('localization', function() {
             .catch(failTest)
             .then(done);
     });
+
+    it('updates ticks and modebar tooltips on Plotly.react', function(done) {
+        Plotly.register({
+            moduleType: 'locale',
+            name: 'xx',
+            dictionary: {Zoom: 'Bigger'},
+            format: {month: '%Y %b'}
+        });
+
+        function getZoomTip() {
+            return gd.querySelector('.modebar-btn[data-val="zoom"]').getAttribute('data-title');
+        }
+
+        plot('en')
+        .then(function() {
+            expect(firstXLabel()).toBe('Jan 2001');
+            expect(getZoomTip()).toBe('Zoom');
+
+            return Plotly.react(gd, gd.data, gd.layout, {locale: 'xx'});
+        })
+        .then(function() {
+            expect(firstXLabel()).toBe('2001 Jan');
+            expect(getZoomTip()).toBe('Bigger');
+
+            // this is discouraged usage, but it works
+            return Plotly.plot(gd, [], {}, {locale: 'en'});
+        })
+        .then(function() {
+            expect(firstXLabel()).toBe('Jan 2001');
+            expect(getZoomTip()).toBe('Zoom');
+        })
+        .catch(failTest)
+        .then(done);
+    });
 });