Skip to content

fix: enable horizontal scroll in Browse table, remove text truncation#5

Open
weidai00 wants to merge 1 commit into
zvec-ai:mainfrom
weidai00:fix/browse-table-scroll
Open

fix: enable horizontal scroll in Browse table, remove text truncation#5
weidai00 wants to merge 1 commit into
zvec-ai:mainfrom
weidai00:fix/browse-table-scroll

Conversation

@weidai00

Copy link
Copy Markdown

Table columns were hidden by overflow:hidden when the collection has
many fields, and long cell text was truncated by max-width:300px and
text-overflow:ellipsis.

Changed .zv-data-table-wrap to overflow-x:auto for horizontal scrolling,
removed max-width on .zv-cell-copy, and allowed text wrapping in
.zv-cell-copy__text.

Signed-off-by: weidai00 m13592398723@163.com

Updated styles for data table and cell copy components.
@feihongxu0824

Copy link
Copy Markdown
Collaborator

Thanks for the fix. Changing .zv-data-table-wrap to overflow-x: auto makes sense and addresses the hidden columns when a collection contains many fields.

One concern: removing max-width entirely may cause long unbroken values—such as JSON, URLs, Base64 strings, or IDs—to make a single column extremely wide. white-space: normal does not reliably wrap these values because they may not contain valid break points.

Could we keep a reasonable cell width and allow arbitrary text wrapping instead? For example:

.zv-cell-copy {
  max-width: 300px;
}

.zv-cell-copy__text {
  min-width: 0;
  white-space: normal;
  overflow-wrap: anywhere;
}

This preserves horizontal scrolling for tables with many columns while still displaying complete cell content without ellipsis.

These styles are also shared by the Browse table and the Query results table, so please verify the behavior on both pages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants