Skip to content

Conversation

Anil567849
Copy link
Contributor

Summary

Fixes the issue where long, non-breakable words in alert messages would overflow the container.

Motivation and Context

Previously, if an alert contained a long string without spaces, it would extend outside the container, breaking the UI. This change ensures proper word wrapping or breaking to keep the alert content inside the container.

How Has This Been Tested?

  • Tested alerts with normal messages.
  • Tested alerts with long, unbroken strings.
  • Verified UI remains intact across different screen sizes.

Breaking Changes

No breaking changes. Existing alert functionality remains the same.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

This fix is proactive; no issue was reported. It improves alert component resilience against unexpected input.

Screenshots:
Before:
image

After:
image

Copy link
Member

@cliffhall cliffhall left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@cliffhall cliffhall enabled auto-merge October 10, 2025 16:11
@cliffhall cliffhall merged commit d85d20f into modelcontextprotocol:main Oct 10, 2025
5 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.

2 participants