GridColumn to only apply displayWidth to multiLine columns #1900

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

cnathe merged 6 commits into develop from fb_displayWidthMultiLine

Nov 25, 2025

packages/components/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

packages/components/package.json

-Original file line number
+Diff line change
@@ -1,6 +1,6 @@
     {
       "name": "@labkey/components",
-      "version": "6.72.0",
+      "version": "6.72.1",
       "description": "Components, models, actions, and utility functions for LabKey applications and pages",
       "sideEffects": false,
       "files": [
@@ Expand Down @@

packages/components/releaseNotes/components.md

-Original file line number
+Diff line change
@@ -1,6 +1,10 @@
     # @labkey/components
     Components, models, actions, and utility functions for LabKey applications and pages
+    ###  version 6.72.1
+    *Released*: 25 November 2025
+    - QueryColumn to only apply displayWidth for multiLine columns
     ###  version 6.72.0
     *Released*: 20 November 2025
     - Default to detailed audit behavior
@@ Expand Down @@

packages/components/src/internal/components/base/Grid.tsx

-Original file line number
+Diff line change
@@ Expand Up @@
                     raw: c,
                     tableCell: c.tableCell,
                     title: c.title || c.caption,
-                    width: c.width || c.displayWidth,
+                    width: c.width,
                 });
             })
             .toList();
@@ Expand Down @@

packages/components/src/public/IQueryColumn.ts

-Original file line number
+Diff line change
@@ Expand Up / @@ -18,6 +18,7 @@ export interface IQueryColumn { @@
         displayField?: string;
         displayFieldJsonType?: string;
         displayFieldSqlType?: string;
+        displayWidth: number | string;
         // excludeFromShifting: boolean;
         // ext: any;
         facetingBehaviorType: string;
@@ Expand Down @@

packages/components/src/public/QueryColumn.ts

-Original file line number
+Diff line change
@@ Expand Up / @@ -205,6 +205,11 @@ export class QueryColumn implements IQueryColumn { @@
             if (rawColumn && rawColumn.lookup !== undefined) {
                 Object.assign(this, { lookup: new QueryLookup(rawColumn.lookup) });
             }
+            // only apply the displayWidth, if available, to multiLine text columns for now
+            if (rawColumn?.displayWidth && this.width === undefined && this.rangeURI?.endsWith('multiLine')) {
+                Object.assign(this, { width: rawColumn.displayWidth });
+            }
         }
         static ANCESTORS_PREFIX = 'Ancestors';
@@ Expand Down @@

Provide feedback