Move legendonly restriction to supplyTraceDefaults

Author: rreusser

src/components/legend/draw.js

@@ -510,8 +510,9 @@ function handleClick(g, gd, numClicks) {
 
         for(i = 0; i < fullData.length; i++) {
             allTraces.push(i);
-            // For carpet traces in particular, push the current visibility. For others, legendonly:
-            traceVisibility.push(Registry.traceIs(fullData[i], 'notLegendIsolatable') ? fullData[i].visible : 'legendonly');
+            // Allow the legendonly state through for *all* trace types (including
+            // carpet for which it's overridden with true/false in supplyDefaults)
+            traceVisibility.push('legendonly');
         }
 
         if(legendgroup === '') {

src/plots/plots.js

@@ -860,6 +860,12 @@ plots.supplyTraceDefaults = function(traceIn, traceOutIndex, layout, traceInInde
             coerce('legendgroup');
         }
 
+        if(plots.traceIs(traceOut, 'notLegendIsolatable')) {
+            // This clears out the legendonly state for traces like carpet that
+            // cannot be isolated in the legend
+            traceOut.visible = !!traceOut.visible;
+        }
+
         plots.supplyTransformDefaults(traceIn, traceOut, layout);
     }