Merge pull request #652 from plotly/textinfo

TextInfo tweak for pies

Author: nicolaskruchten

src/components/fields/derived.js

@@ -482,6 +482,23 @@ export const TextPosition = connectToContainer(UnconnectedDropdown, {
   },
 });
 
+export const TextInfo = connectToContainer(UnconnectedFlaglist, {
+  modifyPlotProps: (props, context, plotProps) => {
+    const {localize: _, container} = context;
+    const options = [
+      {label: _('Label'), value: 'label'},
+      {label: _('Value'), value: 'value'},
+      {label: _('%'), value: 'percent'},
+    ];
+
+    if (container.text) {
+      options.push({label: _('Text'), value: 'text'});
+    }
+
+    plotProps.options = options;
+  },
+});
+
 export const HoverInfo = connectToContainer(UnconnectedFlaglist, {
   modifyPlotProps: (props, context, plotProps) => {
     const {localize: _, container} = context;

src/default_panels/StyleTracesPanel.js

@@ -35,6 +35,7 @@ import {
   BinningDropdown,
   NumericReciprocal,
   ShowInLegend,
+  TextInfo,
 } from '../components/fields/derived';
 
 const StyleTracesPanel = (props, {localize: _}) => (
@@ -94,17 +95,6 @@ const StyleTracesPanel = (props, {localize: _}) => (
       />
     </PlotlySection>
 
-    <PlotlySection name={_('Text Attributes')}>
-      <Flaglist
-        attr="textinfo"
-        options={[
-          {label: _('Label'), value: 'label'},
-          {label: _('Text'), value: 'text'},
-          {label: _('Value'), value: 'value'},
-          {label: _('%'), value: 'percent'},
-        ]}
-      />
-    </PlotlySection>
     <PlotlySection name={_('Header')}>
       <Numeric label={_('Height')} attr="header.height" />
       <MultiColorPicker label={_('Fill Color')} attr="header.fill.color" />
@@ -346,6 +336,9 @@ const StyleTracesPanel = (props, {localize: _}) => (
       <FillDropdown attr="fill" label={_('Fill to')} />
       <MultiColorPicker label={_('Color')} attr="fillcolor" />
     </PlotlySection>
+    <PlotlySection name={_('Text Attributes')}>
+      <TextInfo attr="textinfo" />
+    </PlotlySection>
     <TraceTypeSection
       name={_('Text')}
       traceTypes={[