Skip to content

don't truncate question#312220

Merged
meganrogge merged 2 commits intomainfrom
merogge/question-truncate
Apr 23, 2026
Merged

don't truncate question#312220
meganrogge merged 2 commits intomainfrom
merogge/question-truncate

Conversation

@meganrogge
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 23, 2026 21:05
@meganrogge meganrogge self-assigned this Apr 23, 2026
@meganrogge meganrogge added this to the 1.118.0 milestone Apr 23, 2026
@meganrogge meganrogge enabled auto-merge (squash) April 23, 2026 21:06
Copy link
Copy Markdown
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

Updates the chat question carousel header styling so long question titles wrap instead of being visually truncated in the chat UI.

Changes:

  • Allow the question header row to grow vertically by removing overflow: hidden.
  • Top-align the title row content so action buttons don’t vertically center against multi-line titles.
  • Adjust title wrapping rules (word-break / overflow-wrap / white-space) to permit multi-line titles and break very long tokens.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/media/chatQuestionCarousel.css Updates header/title layout and wrapping to avoid truncating long carousel questions.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: dcc7279e Current: 610e05d4

Changed (10)

chat/chatQuestionCarousel/SingleTextQuestion/Dark
Before After
before after
chat/chatQuestionCarousel/SingleTextQuestion/Light
Before After
before after
chat/chatQuestionCarousel/SingleSelectQuestion/Dark
Before After
before after
chat/chatQuestionCarousel/SingleSelectQuestion/Light
Before After
before after
chat/chatQuestionCarousel/MultiSelectQuestion/Dark
Before After
before after
chat/chatQuestionCarousel/MultiSelectQuestion/Light
Before After
before after
chat/chatQuestionCarousel/MultipleQuestions/Dark
Before After
before after
chat/chatQuestionCarousel/MultipleQuestions/Light
Before After
before after
chat/chatQuestionCarousel/NoSkip/Dark
Before After
before after
chat/chatQuestionCarousel/NoSkip/Light
Before After
before after

@meganrogge meganrogge merged commit 17813fd into main Apr 23, 2026
30 checks passed
@meganrogge meganrogge deleted the merogge/question-truncate branch April 23, 2026 21:42
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