Add block.hideLabel option

Resolves #71.
jakezatecky · Jun 9, 2017 · 854f858 · 854f858
1 parent 30adf3a
commit 854f858
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/d3-funnel/D3Funnel.js b/src/d3-funnel/D3Funnel.js
@@ -316,6 +316,7 @@ class D3Funnel {
 				height: this.settings.height * ratio,
 				fill: this.colorizer.getBlockFill(block.backgroundColor, index, this.settings.fillType),
 				label: {
+					enabled: !block.hideLabel,
 					raw: block.label,
 					formatted: this.labelFormatter.format(block),
 					color: this.colorizer.getLabelColor(block.labelColor),
@@ -766,7 +767,7 @@ class D3Funnel {
 			});
 		}
 
-		if (this.settings.label.enabled) {
+		if (this.settings.label.enabled && block.label.enabled) {
 			this.addBlockLabel(group, index);
 		}
 	}

diff --git a/test/d3-funnel/D3Funnel.js b/test/d3-funnel/D3Funnel.js
@@ -157,6 +157,19 @@ describe('D3Funnel', () => {
 				assert.equal('Node C: Three', d3.select(texts[2]).text());
 			});
 
+			it('should hide the labels of any row specified', () => {
+				getFunnel().draw([
+					{ label: 'Node A', value: 1, hideLabel: true },
+					{ label: 'Node B', value: 2 },
+					{ label: 'Node C', value: 3, hideLabel: true },
+				]);
+
+				const texts = getSvg().selectAll('text').nodes();
+
+				assert.equal('Node B: 2', d3.select(texts[0]).text());
+				assert.equal(undefined, texts[1]);
+			});
+
 			it('should use colors assigned to a data element', () => {
 				getFunnel().draw([
 					['A', 1, '#111'],