Skip to content

Preview title wrap - #7918

Open
ljowen wants to merge 1 commit into
mainfrom
preview-title-wrap
Open

Preview title wrap#7918
ljowen wants to merge 1 commit into
mainfrom
preview-title-wrap

Conversation

@ljowen

@ljowen ljowen commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

What this PR does

Fix data preview title not wrapping long unbroken names (e.g. URLs), which caused a horizontal scrollbar in the preview panel and embedded iframes.

Test me

Preview a dataset with a very long name, it should now wrap instead of overflowing with horizontal scrollbar

Before
Screenshot 2026-08-20 at 1 44 50 pm

After
Screenshot 2026-08-20 at 1 43 08 pm

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

@ljowen
ljowen force-pushed the preview-title-wrap branch from 96f1c53 to 62c05a5 Compare August 20, 2026 03:55
@soyarsauce

Copy link
Copy Markdown
Collaborator

lgtm though FWIW we have a sync issue with the repos right now.

image

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