From 44277c0001c745293b9e93006d6c947ed473eced Mon Sep 17 00:00:00 2001
From: Jae Sung Park <alberto.park@gmail.com>
Date: Tue, 8 Mar 2022 18:10:41 +0900
Subject: [PATCH] fix(axis): Fix culling visibility on dynamic loading

Make text node's visibility to be updated from the possible
visibility status.

Ref #2582
---
 src/ChartInternal/Axis/Axis.ts | 12 +++---
 test/internals/axis-spec.ts    | 73 ++++++++++++++++++++++++++++++++++
 2 files changed, 80 insertions(+), 5 deletions(-)

diff --git a/src/ChartInternal/Axis/Axis.ts b/src/ChartInternal/Axis/Axis.ts
index a1ae13581..e6a8ea74b 100644
--- a/src/ChartInternal/Axis/Axis.ts
+++ b/src/ChartInternal/Axis/Axis.ts
@@ -941,6 +941,7 @@ class Axis {
 		const $$ = this.owner;
 		const {config, state: {clip, current}, $el} = $$;
 
+
 		["subX", "x", "y", "y2"].forEach(type => {
 			const axis = $el.axis[type];
 
@@ -966,13 +967,14 @@ class Axis {
 						}
 					}
 
-					tickNodes.each(function(d) {
-						if (tickValues.indexOf(d) % intervalForCulling) {
+					tickNodes
+						.each(function(d) {
 							const node = (lines ? this.querySelector("text") : this);
 
-							node && (node.style.display = "none");
-						}
-					});
+							if (node) {
+								node.style.display = tickValues.indexOf(d) % intervalForCulling ? "none" : null;
+							}
+						});
 				} else {
 					tickNodes.style("display", null);
 				}
diff --git a/test/internals/axis-spec.ts b/test/internals/axis-spec.ts
index 9b5c09470..c4d04b71b 100644
--- a/test/internals/axis-spec.ts
+++ b/test/internals/axis-spec.ts
@@ -2627,6 +2627,79 @@ describe("AXIS", function() {
 				expect(axis[v].selectAll(selector).size()).to.be.equal(args.axis[v].tick.culling.max);
 			});
 		});
+
+		it("set options", () => {
+			args = {
+				data: {
+					x: "periods",
+					xFormat: "%Y",
+					type: "line",
+					columns: [
+						["periods", "2011", "2012", "2013", "2014", "2015","2016"],
+						["data1", 30, 200, 100, 170, 150, 250]
+					]
+				},
+			  axis: {
+				x: {
+				  tick: {
+					culling: {
+					  max: 8
+					}
+				  }
+				}
+			  }
+			}
+		});
+
+		it("tick text's culling visibility should work correctly", done => {
+			// count visibility text nodes
+			const countVisibility = () => {
+				let visible = 0;
+				let hidden = 0;
+
+				chart.internal.$el.axis.x.selectAll(".tick text")
+					.each(function() {
+						if (this.style.display === "none") {
+							hidden++;
+						} else {
+							visible++;
+						}
+					});
+
+				return {visible, hidden};
+			};
+
+			new Promise((resolve, reject) => {
+				// load new dataset
+				chart.load({
+					columns: [
+						["periods", "1999", "2000", "2001", "2002", "2003","2004","2005", "2006", "2007", "2008", "2009", "2010", "2011", "2012", "2013", "2014", "2015","2016"],
+						["data1", 30, 200, 100, 170, 150, 250, 30, 200, 100, 170, 150, 250, 30, 200, 100, 170, 150, 250]
+					],
+					done: resolve
+				});
+			}).then(() => {
+				const {visible, hidden} = countVisibility();
+
+				expect(visible).to.equal(6);
+				expect(hidden).to.equal(12);				
+
+				return new Promise((resolve, reject) => {
+					// revert to original dataset
+					chart.load({
+						columns: args.data.columns,
+						done: resolve
+					});
+				});
+			}).then(() => {
+				const {visible, hidden} = countVisibility();
+
+				expect(visible).to.equal(6);
+				expect(hidden).to.equal(0);	
+
+				done();
+			});
+		});
 	});
 	
 	describe("Axes tick padding", () => {