Skip to content

ECHOES-1301 Change ToastContent to stretch#677

Merged
gregaubert merged 1 commit intomainfrom
greg/update-toast-body
May 6, 2026
Merged

ECHOES-1301 Change ToastContent to stretch#677
gregaubert merged 1 commit intomainfrom
greg/update-toast-body

Conversation

@gregaubert
Copy link
Copy Markdown
Member

Part of

@netlify
Copy link
Copy Markdown

netlify Bot commented May 5, 2026

Deploy Preview for echoes-react ready!

Name Link
🔨 Latest commit 1204e6d
🔍 Latest deploy log https://app.netlify.com/projects/echoes-react/deploys/69fa07b9072a3e00083a490d
😎 Deploy Preview https://deploy-preview-677--echoes-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown

hashicorp-vault-sonar-prod Bot commented May 5, 2026

ECHOES-1301

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha Bot commented May 5, 2026

Summary

This PR adds a single CSS property to the ToastContent component: align-self: stretch. This makes toast messages expand to fill the full width of their parent container rather than shrinking to content width. The change is minimal and focused on improving the toast visual layout.

What reviewers should know

What to review:

  • The change is in src/common/components/Toast.tsx on the ToastContent styled component (around line 212)
  • Only one line added: align-self: stretch;

Context:

  • This is a styling-only change that affects the cross-axis alignment in a flex container
  • The impact is visual: toasts will now span the full width of their container instead of being constrained to content width
  • No logic changes, component behavior, or API modifications

Testing notes:

  • Visual verification of toast appearance across different message lengths and screen sizes would be helpful
  • Check that toast content (text, icons, buttons) still aligns correctly with the full-width behavior

  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

Copy link
Copy Markdown

@sonar-review-alpha sonar-review-alpha Bot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

Clean, minimal change. ToastContent is a child of ToastBody which has align-items: flex-start, so without align-self: stretch, ToastContent shrinks to its content width. The fix is correct and well-targeted.

🗣️ Give feedback

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next Bot commented May 5, 2026

@gregaubert gregaubert merged commit a4b490e into main May 6, 2026
13 checks passed
@gregaubert gregaubert deleted the greg/update-toast-body branch May 6, 2026 08:09
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