Skip to content

Updating bottom sheet corner radius#2194

Merged
joannaquu merged 5 commits intomicrosoft:mainfrom
JaquelineGomez:user/jagomez/updatingBottomSheetCornerRadius
Jul 31, 2025
Merged

Updating bottom sheet corner radius#2194
joannaquu merged 5 commits intomicrosoft:mainfrom
JaquelineGomez:user/jagomez/updatingBottomSheetCornerRadius

Conversation

@JaquelineGomez
Copy link
Copy Markdown
Contributor

@JaquelineGomez JaquelineGomez commented Jul 30, 2025 •
edited
Loading

Platforms Impacted

  • iOS
  • visionOS
  • macOS

Description of changes

  • Added a new corner radius token
  • Changed the corner radius of the BottomSheets to the new corner radius only if they are using the new iOS version that will be released soon
  • This new corner radius makes the BottomSheets more consistent with Apple styling

Verification

Ran it on two iPhone simulators:

  1. Running on iOS 18
  2. Running on iOS 26

Ran it on two iPad simulators:

  1. Running on iOS 18
  2. Running on iOS 26

The change was only seen on the iPhone and iPad running on iOS26.

Visual Verification
Before After
image image
image image

Pull request checklist

This PR has considered:

  • Light and Dark appearances
  • iOS supported versions (all major versions greater than or equal current target deployment version)
  • VoiceOver and Keyboard Accessibility
  • Internationalization and Right to Left layouts
  • Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
  • SwiftUI consumption (validation or new demo scenarios needed)
Microsoft Reviewers: Open in CodeFlow

@JaquelineGomez JaquelineGomez requested a review from a team as a code owner July 30, 2025 18:53
@JaquelineGomez
Copy link
Copy Markdown
Contributor Author

JaquelineGomez commented Jul 30, 2025

@microsoft-github-policy-service agree company="Microsoft"

anandrajeswaran
anandrajeswaran reviewed Jul 30, 2025
View reviewed changes
joannaquu
joannaquu approved these changes Jul 30, 2025
View reviewed changes
@joannaquu joannaquu merged commit 7b7a8a8 into microsoft:main Jul 31, 2025
7 checks passed
@joannaquu joannaquu mentioned this pull request Oct 23, 2025
Merged
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mischreiber mischreiber mischreiber left review comments

@anandrajeswaran anandrajeswaran anandrajeswaran left review comments

@nodes11 nodes11 nodes11 approved these changes

@joannaquu joannaquu joannaquu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@JaquelineGomez @mischreiber @nodes11 @anandrajeswaran @joannaquu