Merge pull request #65 from RevealBi/text-view-change-spec-type

Updated TextViewVisualization DataSpec type
RevealBi · Sep 26, 2024 · c2be041 · c2be041
2 parents 2d7fa8c + 806e420
commit c2be041
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/src/Reveal.Sdk.Dom/Visualizations/TextViewVisualization.cs b/src/Reveal.Sdk.Dom/Visualizations/TextViewVisualization.cs
@@ -27,9 +27,22 @@ public TextViewVisualization(DataSourceItem dataSourceItem) : this(null, dataSou
         public TextViewVisualization(string title, DataSourceItem dataSourceItem) : base(title, dataSourceItem) { ChartType = ChartType.TextView; }
 
         [JsonProperty(Order = 7)]
-        GridVisualizationDataSpec VisualizationDataSpec { get; set; } = new GridVisualizationDataSpec();
+        VisualizationDataSpec VisualizationDataSpec { get; set; } = new GridVisualizationDataSpec();
 
         [JsonIgnore]
-        public List<TabularColumn> Columns { get { return VisualizationDataSpec.Columns; } }
+        public List<TabularColumn> Columns
+        {
+            get
+            {
+                if (VisualizationDataSpec is GridVisualizationDataSpec gds)
+                {
+                    return gds.Columns;
+                }
+                else
+                {
+                    return new List<TabularColumn>();
+                }
+            }
+        }
     }
 }