Skip to content

feat: truncate long strings data browsing VSCODE-745#1252

Open
tculig wants to merge 8 commits intomongodb-js:mainfrom
tculig:VSCODE-745-truncate-long-strings-data-browsing
Open

feat: truncate long strings data browsing VSCODE-745#1252
tculig wants to merge 8 commits intomongodb-js:mainfrom
tculig:VSCODE-745-truncate-long-strings-data-browsing

Conversation

@tculig
Copy link
Copy Markdown
Contributor

@tculig tculig commented Feb 23, 2026

Description

https://jira.mongodb.org/browse/VSCODE-745

image image

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@tculig tculig changed the title Vscode 745 truncate long strings data browsing feat: truncate long strings data browsing VSCODE-745 Feb 23, 2026
@tculig tculig force-pushed the VSCODE-745-truncate-long-strings-data-browsing branch from 226406b to 94682a1 Compare March 1, 2026 21:07
@tculig tculig marked this pull request as ready for review March 1, 2026 22:53
@tculig tculig requested a review from a team as a code owner March 1, 2026 22:53
@gagik
Copy link
Copy Markdown
Contributor

gagik commented Mar 2, 2026

could we use

text-overflow: ellipsis;

instead?

https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Properties/text-overflow

@lerouxb
Copy link
Copy Markdown
Member

lerouxb commented Mar 2, 2026

When you copy to clipboard it should copy the whole document, not the truncated strings.

truncationMapRef.current,
expandedPathsRef.current,
);
const formatted = toJSString(truncated) ?? '';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember that we also use documentString in handleCopy below. And copy should copy the whole document.

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.

3 participants