Skip to content

[dev-v5][Toast] Fix toast sizing in Safari on macOS and iOS#4871

Merged
vnbaaij merged 1 commit into
microsoft:dev-v5from
MarvinKlein1508:safari-toast-fix
May 18, 2026
Merged

[dev-v5][Toast] Fix toast sizing in Safari on macOS and iOS#4871
vnbaaij merged 1 commit into
microsoft:dev-v5from
MarvinKlein1508:safari-toast-fix

Conversation

@MarvinKlein1508
Copy link
Copy Markdown
Collaborator

@MarvinKlein1508 MarvinKlein1508 commented May 18, 2026

Pull Request

📖 Description

This PR fixes sizing issues on iOS and macOS for FluentToast by setting height: auto.

Before desktop:
grafik

After desktop:

grafik

Before mobile:

grafik

After mobile:

grafik

✅ Checklist

General

  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

  • I have added a new component
  • I have added Unit Tests for my new component
  • I have modified an existing component
  • I have validated the Unit Tests for an existing component

⏭ Next Steps

I have tested this changes on Safari on macOS and iOS and on Chrome, Edge and Firefox. I also tested the positioning. You still need to test for Android.

Copilot AI review requested due to automatic review settings May 18, 2026 11:41
@MarvinKlein1508 MarvinKlein1508 added this to the v5.0-RC3 milestone May 18, 2026
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

Fixes Safari (macOS/iOS) sizing issue for FluentToast by adding height: auto to the toast CSS.

Changes:

  • Add height: auto to the toast styles in FluentToast.ts

@vnbaaij vnbaaij merged commit 4dcf8e2 into microsoft:dev-v5 May 18, 2026
6 of 7 checks passed
@MarvinKlein1508 MarvinKlein1508 deleted the safari-toast-fix branch May 18, 2026 13:32
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.

4 participants