From 6e9cb40e81b62ad3472367e42da6975034abbba0 Mon Sep 17 00:00:00 2001
From: Shumpei Shiina <s417.lama@gmail.com>
Date: Wed, 3 Mar 2021 15:16:05 +0900
Subject: [PATCH] small fixes for pattern fill

---
 src/components/drawing/index.js  | 6 +++---
 src/components/legend/style.js   | 6 +++---
 src/traces/bar/style_defaults.js | 2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/components/drawing/index.js b/src/components/drawing/index.js
index 603673996e1..11ab60dc938 100644
--- a/src/components/drawing/index.js
+++ b/src/components/drawing/index.js
@@ -486,7 +486,7 @@ drawing.pattern = function(sel, gd, patternID, shape, bgcolor, fgcolor, size, so
 
     var pattern = fullLayout._defs.select('.patterns')
         .selectAll('#' + fullID)
-        .data([shape + ';' + bgcolor + ';' + fgcolor + ';' + size + ';' + solidity]);
+        .data([shape + ';' + bgcolor + ';' + fgcolor + ';' + size + ';' + solidity], Lib.identity);
 
     pattern.exit().remove();
 
@@ -692,8 +692,8 @@ drawing.singlePointStyle = function(d, sel, trace, fns, gd) {
                 [[0, gradientColor], [1, fillColor]], 'fill');
         } else if(patternShape) {
             var patternBGColor = getPatternAttr(markerPattern.bgcolor, d.i, null);
-            var patternSize = getPatternAttr(markerPattern.size, d.i, 1);
-            var patternSolidity = getPatternAttr(markerPattern.solidity, d.i, 1);
+            var patternSize = getPatternAttr(markerPattern.size, d.i, 8);
+            var patternSolidity = getPatternAttr(markerPattern.solidity, d.i, 0.3);
             var perPointPattern = Array.isArray(markerPattern.shape) ||
                                   Array.isArray(markerPattern.bgcolor) ||
                                   Array.isArray(markerPattern.size) ||
diff --git a/src/components/legend/style.js b/src/components/legend/style.js
index 1895d84630b..6b58690fe9a 100644
--- a/src/components/legend/style.js
+++ b/src/components/legend/style.js
@@ -373,9 +373,9 @@ module.exports = function style(s, gd, legend) {
 
             if(patternShape) {
                 var patternBGColor = getPatternAttr(markerPattern.bgcolor, null);
-                var patternSize = getPatternAttr(markerPattern.size, 1);
-                var patternSolidity = getPatternAttr(markerPattern.solidity, 1);
-                var patternID = trace.uid;
+                var patternSize = Math.min(12, getPatternAttr(markerPattern.size, 8));
+                var patternSolidity = getPatternAttr(markerPattern.solidity, 0.3);
+                var patternID = 'legend-' + trace.uid;
                 p.call(Drawing.pattern, gd, patternID, patternShape, patternBGColor,
                        fillColor, patternSize, patternSolidity, 'fill');
             } else {
diff --git a/src/traces/bar/style_defaults.js b/src/traces/bar/style_defaults.js
index 60223ab4d12..43671373958 100644
--- a/src/traces/bar/style_defaults.js
+++ b/src/traces/bar/style_defaults.js
@@ -24,7 +24,7 @@ module.exports = function handleStyleDefaults(traceIn, traceOut, coerce, default
     coerce('marker.line.width');
     coerce('marker.opacity');
     var pattern = coerce('marker.pattern.shape');
-    if (pattern) {
+    if(pattern) {
         coerce('marker.pattern.bgcolor');
         coerce('marker.pattern.size');
         coerce('marker.pattern.solidity');