revert changes to modebar.js for dynamic background

Author: archmoj

src/components/modebar/modebar.js

@@ -4,7 +4,6 @@ var d3 = require('@plotly/d3');
 var isNumeric = require('fast-isnumeric');
 
 var Lib = require('../../lib');
-var Color = require('../color');
 var Icons = require('../../fonts/ploticon');
 var version = require('../../version').version;
 
@@ -63,12 +62,6 @@ proto.update = function(graphInfo, buttons) {
     Lib.addRelatedStyleRule(modeBarId, '#' + modeBarId + ' .modebar-btn:hover .icon path', 'fill: ' + style.activecolor);
     Lib.addRelatedStyleRule(modeBarId, '#' + modeBarId + ' .modebar-btn.active .icon path', 'fill: ' + style.activecolor);
 
-    // logo constrast color
-    var logoId = modeBarId + '-logo';
-    var logoSelector = context.displayModeBar === 'hover' ? '.plotlyjsicon' : '';
-    Lib.deleteRelatedStyleRule(logoId);
-    Lib.addRelatedStyleRule(logoId, logoSelector + ' .cls-bg-contrast', 'fill: ' + Color.contrast(fullLayout.paper_bgcolor));
-
     // if buttons or logo have changed, redraw modebar interior
     var needsNewButtons = !this.hasButtons(buttons);
     var needsNewLogo = (this.hasLogo !== context.displaylogo);
@@ -82,7 +75,7 @@ proto.update = function(graphInfo, buttons) {
         this.updateButtons(buttons);
 
         if(context.watermark || context.displaylogo) {
-            var logoGroup = this.getLogo(graphInfo);
+            var logoGroup = this.getLogo();
             if(context.watermark) {
                 logoGroup.className = logoGroup.className + ' watermark';
             }
@@ -299,7 +292,7 @@ function jsVersion(str) {
 /**
  * @return {HTMLDivElement} The logo image wrapped in a group
  */
-proto.getLogo = function(graphInfo) {
+proto.getLogo = function() {
     var group = this.createGroup();
     var a = document.createElement('a');
 
@@ -310,15 +303,6 @@ proto.getLogo = function(graphInfo) {
 
     a.appendChild(this.createIcon(Icons.newplotlylogo));
 
-    var context = graphInfo._context;
-    var fullLayout = graphInfo._fullLayout;
-    var modeBarId = 'modebar-' + fullLayout._uid;
-
-    var logoId = modeBarId + '-logo';
-    var logoSelector = context.displayModeBar === 'hover' ? '.plotlyjsicon' : '';
-    Lib.deleteRelatedStyleRule(logoId);
-    Lib.addRelatedStyleRule(logoId, logoSelector + ' .cls-bg-contrast', 'fill: ' + Color.contrast(fullLayout.paper_bgcolor));
-
     group.appendChild(a);
     return group;
 };
@@ -334,7 +318,6 @@ proto.removeAllButtons = function() {
 proto.destroy = function() {
     Lib.removeElement(this.container.querySelector('.modebar'));
     Lib.deleteRelatedStyleRule(this._uid);
-    Lib.deleteRelatedStyleRule(this._uid + '-logo');
 };
 
 function createModeBar(gd, buttons) {