Skip to content

Comments

Fix tabular data column text wrapping#199

Draft
adrianlyjak wants to merge 1 commit intomainfrom
cursor/LI-4645-fix-tabular-data-column-text-wrapping-3d82
Draft

Fix tabular data column text wrapping#199
adrianlyjak wants to merge 1 commit intomainfrom
cursor/LI-4645-fix-tabular-data-column-text-wrapping-3d82

Conversation

@adrianlyjak
Copy link
Collaborator

Normalize column widths and enable text wrapping in the extracted data table grid to prevent layout overflow issues.

The previous implementation allowed long text, especially file names, to overflow columns, leading to unpredictable table layouts. This PR introduces CSS classes to ensure text wraps within defined column widths, maintaining a consistent and readable table structure.


Linear Issue: LI-4645

Open in Cursor Open in Web

Co-authored-by: adrian <adrian@runllama.ai>
@cursor
Copy link

cursor bot commented Dec 3, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@changeset-bot
Copy link

changeset-bot bot commented Dec 3, 2025

⚠️ No Changeset found

Latest commit: 746a1a3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@yashi-runllamaindex
Copy link

@adrianlyjak - you close this PR but I am eager to know if are we not merging this ?

@adrianlyjak
Copy link
Collaborator Author

@yashi-runllamaindex this cursor implementation doesn’t look very good or general. I think better to start from scratch

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