Adjust rendering of legend to resolve image baselines

It appears the element marked with `legendtoggle` contributes visually,
which means it should not be skipped over when rendering in order to
preserve the existing tests for image baselines.

Fixes #5177

Author: miqh

src/components/legend/draw.js

@@ -460,18 +460,19 @@ function ensureLength(str, maxLength) {
 }
 
 function setupTraceToggle(g, gd) {
-    if(gd._context.staticPlot) return;
-
     var doubleClickDelay = gd._context.doubleClickDelay;
     var newMouseDownTime;
     var numClicks = 1;
 
     var traceToggle = Lib.ensureSingle(g, 'rect', 'legendtoggle', function(s) {
-        s.style('cursor', 'pointer')
-            .attr('pointer-events', 'all')
-            .call(Color.fill, 'rgba(0,0,0,0)');
+        if(!gd._context.staticPlot) {
+            s.style('cursor', 'pointer').attr('pointer-events', 'all');
+        }
+        s.call(Color.fill, 'rgba(0,0,0,0)');
     });
 
+    if(gd._context.staticPlot) return;
+
     traceToggle.on('mousedown', function() {
         newMouseDownTime = (new Date()).getTime();
         if(newMouseDownTime - gd._legendMouseDownTime < doubleClickDelay) {