Skip to content

Update widget previewer documentation for dependencies#13306

Merged
johnpryan merged 2 commits intomainfrom
update-ffi-widget-preview-warning
Apr 24, 2026
Merged

Update widget previewer documentation for dependencies#13306
johnpryan merged 2 commits intomainfrom
update-ffi-widget-preview-warning

Conversation

@johnpryan
Copy link
Copy Markdown
Contributor

See flutter/skills#89 for details

Consolidate information about widgets with transitive dependencies on dart:io and dart:ffi.
@johnpryan johnpryan requested a review from bkonyi April 24, 2026 17:12
@johnpryan johnpryan marked this pull request as ready for review April 24, 2026 17:13
@johnpryan johnpryan requested review from a team and sfshaza2 as code owners April 24, 2026 17:13
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 updates the widget previewer documentation to clarify that widgets with transitive dependencies on dart:ffi now load correctly but throw exceptions when invoked, matching the behavior of dart:io. The review feedback suggests refining the phrasing for better readability, removing a trailing whitespace character, and deleting an orphaned link reference resulting from the text changes.

Comment thread src/content/tools/widget-previewer.md Outdated
@flutter-website-bot
Copy link
Copy Markdown
Collaborator

flutter-website-bot commented Apr 24, 2026

Visit the preview URL for this PR (updated for commit 505ac47):

https://flutter-docs-prod--pr13306-update-ffi-widget-preview-w-k1y0qpl7.web.app

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@johnpryan johnpryan merged commit 524f939 into main Apr 24, 2026
10 checks passed
@johnpryan johnpryan deleted the update-ffi-widget-preview-warning branch April 24, 2026 20:53
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