add a few TODO comments related to polar.hole

Author: etpinard

src/plots/polar/constants.js

@@ -24,6 +24,9 @@ module.exports = {
         'frontplot',
         'angular-axis',
         'radial-axis',
+        // TODO hmm, putting angular-line above radial-axis
+        // can sometimes hide radial ticks and label
+        // this happens especially for the 'inner' angular axis
         'angular-line',
         'radial-line'
     ],

src/plots/polar/polar.js

@@ -597,6 +597,9 @@ proto.updateAngularAxis = function(fullLayout, polarLayout) {
     }
     _this.vangles = vangles;
 
+    // TODO maybe two arcs is better here?
+    // maybe split style attributes between inner and outer angular axes?
+
     updateElement(layers['angular-line'].select('path'), angularLayout.showline, {
         d: _this.pathSubplot(),
         transform: strTranslate(cx, cy)
@@ -937,6 +940,8 @@ proto.updateRadialDrag = function(fullLayout) {
     var tx = cx + (radius + bl2) * Math.cos(angle0);
     var ty = cy - (radius + bl2) * Math.sin(angle0);
 
+    // TODO add 'inner' drag box when innerRadius > 0 !!
+
     d3.select(radialDrag)
         .attr('transform', strTranslate(tx, ty));