Skip to content

fix: wrap long docstrings in parameter hints widget#292258

Merged
mjbvz merged 2 commits intomicrosoft:mainfrom
murataslan1:fix/parameter-hints-word-wrap
Mar 11, 2026
Merged

fix: wrap long docstrings in parameter hints widget#292258
mjbvz merged 2 commits intomicrosoft:mainfrom
murataslan1:fix/parameter-hints-word-wrap

Conversation

@murataslan1
Copy link
Contributor

Summary

Fixes long docstrings in the parameter hints widget not wrapping. Previously, long strings were displayed on a single line with only a vertical scroll bar, making it impossible to read long descriptions.

Changes

  • Add overflow-wrap: break-word and word-break: break-word to .docs so long strings wrap within the widget width
  • Add min-width: 0 to .docs for proper flexbox sizing (allows the element to shrink below content size)
  • Add overflow-wrap, word-break, and max-width: 100% to .markdown-docs for rendered markdown content

Before

Long docstrings displayed on a single line, requiring horizontal scroll (but no horizontal scrollbar was shown).

After

Long docstrings wrap within the parameter hints widget (max-width: 440px), making them readable without horizontal scrolling.

Fixes #142888

Made with Cursor

Add overflow-wrap and word-break to the parameter hints docs section
so that long docstrings wrap instead of displaying on a single line
with horizontal scroll. Also add min-width: 0 for proper flexbox
sizing and max-width: 100% for markdown content.

Fixes microsoft#142888

Co-authored-by: Cursor <[email protected]>
Copilot AI review requested due to automatic review settings February 2, 2026 08:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes long docstrings in the parameter hints widget that were not wrapping properly, making them difficult to read.

Changes:

  • Adds CSS text wrapping properties to .docs and .markdown-docs classes to ensure long text wraps within the widget's max-width constraint (440px)
  • Adds min-width: 0 to allow proper flexbox shrinking behavior

@murataslan1
Copy link
Contributor Author

Ready for review. CSS-only change in parameterHints.css — adds overflow-wrap and word-break so long docstrings wrap.

@mjbvz mjbvz enabled auto-merge March 11, 2026 02:13
@mjbvz mjbvz added this to the 1.112.0 milestone Mar 11, 2026
@murataslan1
Copy link
Contributor Author

Hi @mjbvz @TylerLeonhardt — thank you for the approvals! Just wanted to check if there's anything else needed from my side for this to be merged. Happy to make any final adjustments if needed.

@mjbvz mjbvz merged commit ebaff58 into microsoft:main Mar 11, 2026
28 of 29 checks passed
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.

Long parameter hints are not wrapped

4 participants