tweak docs & remove commented-out code

Author: alexcjohnson

src/plots/plots.js

@@ -265,9 +265,9 @@ var extraFormatKeys = [
  *   trace into the new one. Use skipUpdateCalc to defer this (needed by Plotly.react)
  *
  * gd.data, gd.layout:
- *   are precisely what the user specified,
- *   these fields shouldn't be modified nor used directly
- *   after the supply defaults step.
+ *   are precisely what the user specified (except as modified by cleanData/cleanLayout),
+ *   these fields shouldn't be modified (except for filling in some auto values)
+ *   nor used directly after the supply defaults step.
  *
  * gd._fullData, gd._fullLayout:
  *   are complete descriptions of how to draw the plot,

src/traces/contourcarpet/attributes.js

@@ -40,9 +40,6 @@ module.exports = extendFlat({}, {
     atype: heatmapAttrs.xtype,
     btype: heatmapAttrs.ytype,
 
-    // unimplemented - looks like connectgaps is implied true
-    // connectgaps: heatmapAttrs.connectgaps,
-
     fillcolor: contourAttrs.fillcolor,
 
     autocontour: contourAttrs.autocontour,

src/traces/contourcarpet/defaults.js

@@ -55,9 +55,6 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         coerce('text');
         var isConstraint = (coerce('contours.type') === 'constraint');
 
-        // Unimplemented:
-        // coerce('connectgaps', Lib.isArray1D(traceOut.z));
-
         // trace-level showlegend has already been set, but is only allowed if this is a constraint
         if(!isConstraint) delete traceOut.showlegend;