normalize new lines

Author: emrberk

src/components/ResultGrid/ResultGrid.tsx

@@ -26,6 +26,7 @@ import {
   isLeftAligned,
   formatCellValue,
   formatColumnType,
+  toSingleLineDisplay,
 } from "./inlineGridUtils"
 import { useGridKeyboardNav } from "./useGridKeyboardNav"
 import {
@@ -105,7 +106,9 @@ const GridCell = React.memo(function GridCell({
 }: GridCellProps) {
   const colType = col?.type ?? ""
   const align = isLeftAligned(colType) ? "left" : "right"
-  const displayValue = loaded ? formatCellValue(rawValue, col, colWidth) : ""
+  const displayValue = loaded
+    ? toSingleLineDisplay(formatCellValue(rawValue, col, colWidth))
+    : ""
   return (
     <Cell
       id={`cell-${rowIndex}-${colIndex}`}

src/components/ResultGrid/inlineGridUtils.test.ts

@@ -6,6 +6,7 @@ import {
   formatCellValueForCopy,
   formatColumnType,
   isLeftAligned,
+  toSingleLineDisplay,
 } from "./inlineGridUtils"
 import type { ColumnDefinition } from "../../utils/questdb/types"
 
@@ -193,6 +194,34 @@ describe("formatCellValueForCopy", () => {
     // Then the copied text is preserved exactly as returned by the server
     expect(out).toBe("a&b<c>d")
   })
+
+  it("preserves embedded line breaks so copy keeps the raw value", () => {
+    const out = formatCellValueForCopy(
+      "line1\nline2\r\nline3",
+      col("x", "VARCHAR"),
+    )
+
+    expect(out).toBe("line1\nline2\r\nline3")
+  })
+})
+
+describe("toSingleLineDisplay", () => {
+  it("collapses line breaks to a single space so cells render on one line", () => {
+    expect(toSingleLineDisplay("line1\nline2")).toBe("line1 line2")
+    expect(toSingleLineDisplay("line1\r\nline2")).toBe("line1 line2")
+    expect(toSingleLineDisplay("line1\rline2")).toBe("line1 line2")
+    expect(toSingleLineDisplay("a\n\n\nb")).toBe("a b")
+  })
+
+  it("preserves leading and interior spaces (rendered via white-space: pre)", () => {
+    expect(toSingleLineDisplay("    indented")).toBe("    indented")
+    expect(toSingleLineDisplay("a    b")).toBe("a    b")
+  })
+
+  it("leaves values without line breaks untouched", () => {
+    expect(toSingleLineDisplay("a&amp;b<c>d")).toBe("a&amp;b<c>d")
+    expect(toSingleLineDisplay("")).toBe("")
+  })
 })
 
 describe("sampleColumnWidths", () => {

src/components/ResultGrid/inlineGridUtils.ts

@@ -174,3 +174,6 @@ export const formatCellValueForCopy = (
 
   return formatCellValue(value, col)
 }
+
+export const toSingleLineDisplay = (text: string): string =>
+  text.replace(/[\r\n]+/g, " ")

src/js/console/grid.js

@@ -26,6 +26,7 @@ import { toast } from "../../components"
 import { trackEvent } from "../../modules/ConsoleEventTracker"
 import { ConsoleEvent } from "../../modules/ConsoleEventTracker/events"
 import { buildResultPageMarkdown } from "../../components/ResultGrid/resultPageMarkdown"
+import { toSingleLineDisplay } from "../../components/ResultGrid/inlineGridUtils"
 
 const hashString = (str) => {
   let hash = 0
@@ -1106,7 +1107,9 @@ export function grid(rootElement, _paginationFn, id) {
     if (cellData !== null) {
       const layoutEntry = getLayoutEntry()
       const columnWidth = layoutEntry.deviants[column.name] ?? null
-      cell.textContent = getDisplayedCellValue(column, cellData, columnWidth)
+      cell.textContent = toSingleLineDisplay(
+        getDisplayedCellValue(column, cellData, columnWidth),
+      )
 
       cell.classList.remove("qg-null")