Skip to content

Add a CupertinoSheet example#13223

Merged
sfshaza2 merged 6 commits intomainfrom
issue-11598-cupertino-sheet
Mar 26, 2026
Merged

Add a CupertinoSheet example#13223
sfshaza2 merged 6 commits intomainfrom
issue-11598-cupertino-sheet

Conversation

@sfshaza2
Copy link
Copy Markdown
Contributor

Fixes #11598

Note to reviewers: Do we still need/want this?

@sfshaza2 sfshaza2 requested a review from dkwingsmt March 26, 2026 19:56
@sfshaza2 sfshaza2 requested a review from a team as a code owner March 26, 2026 19:56
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new cookbook recipe for displaying Cupertino sheets in Flutter, including a sample application, project configuration, and documentation. The review feedback suggests adopting arrow syntax for single-line functions to align with Effective Dart guidelines and removing the unnecessary Material Design flag from the pubspec file to optimize the Cupertino-specific example.

@flutter-website-bot
Copy link
Copy Markdown
Collaborator

flutter-website-bot commented Mar 26, 2026

Visit the preview URL for this PR (updated for commit 6a4bf04):

https://flutter-docs-prod--pr13223-issue-11598-cupertino-sheet-k56xzcol.web.app

Copy link
Copy Markdown
Contributor

@dkwingsmt dkwingsmt left a comment

Choose a reason for hiding this comment

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

Overall LGTM except for one comment. Thank you!

@sfshaza2 sfshaza2 merged commit d3d69b7 into main Mar 26, 2026
10 checks passed
@sfshaza2 sfshaza2 deleted the issue-11598-cupertino-sheet branch March 26, 2026 23:33
sfshaza2 added a commit that referenced this pull request Mar 26, 2026
Fixes #11598

Note to reviewers: Do we still need/want this?

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

Create a recipe to demonstrate the new Cupertino-Sheet class

4 participants