Change default rendering of arrays in a table cell to display number …

…of array elements
discoveryjs · Oct 17, 2023 · c16c1ba · c16c1ba
1 parent 001f69c
commit c16c1ba
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 - Uniformed font and size for monospace views
 - Enlarged action buttons in `struct` and `structure` views
 - Changed rendering of after-annotations to render in a prelude when a value is expanded
+- Changed default rendering of arrays in a table cell to display number of array elements instead of `[…]`
 - Fixed editor to not render text until gets focus
 - Fixed suggestion popup positioning in query editor in some cases
 - Modified `Widget` to utilize deep comparison for page params, ensuring more accurate detection of changes

diff --git a/src/views/table/table-cell.js b/src/views/table/table-cell.js
@@ -7,8 +7,8 @@ const hasOwnProperty = Object.hasOwnProperty;
 
 function defaultCellRender(el, data, isDataObject) {
     if (Array.isArray(data)) {
-        el.classList.add('complex');
-        el.textContent = data.length ? '[…]' : '[]';
+        el.classList.add('number');
+        el.textContent = data.length || '';
         return;
     }
 
@@ -52,7 +52,7 @@ export default function(host) {
         let { content, details, colSpan, scalarAsStruct } = config;
         const isDataObject =
             data !== null &&
-            typeof data === 'object' &&
+            (Array.isArray(data) ? data.length > 0 : typeof data === 'object') &&
             data instanceof RegExp === false;
 
         if (typeof colSpan === 'number' && colSpan > 1) {