Repository navigation
Conversation
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the Flutter Widget Previewer documentation to include a new section on preview parameters, details on build caching, updated IDE support requirements, and instructions for searching, filtering, and controls. It also revises the code examples to demonstrate custom previews with PreviewThemeData and runtime transformations. Feedback on these changes suggests improving text readability by removing repetitive phrasing, handling potential null values for preview names to avoid displaying 'null' in the UI, and defensively using MediaQuery.maybePlatformBrightnessOf to prevent runtime exceptions when a MediaQuery ancestor is missing.
…ng and null safety
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates
skills/flutter-add-widget-preview/SKILL.mdto align with recent updates to the Flutter Widget Previewer documentation (flutter/website#13704 and flutter/website#13861):PreviewThemeDataexample to extendPreviewThemeData(@override Widget apply(BuildContext context, Widget child)) and callbuilder.build()instead of the removedPreviewThemeData(materialLight: ..., materialDark: ...)constructor.@Previewparameter reference (name,group,size,textScaleFactor,wrapper,theme,brightness,localizations)..widget_preview/build caching.Note: This is AI-generated (I'm testing an automation to try to update skills based on doc updates). I looked at the suggestions and it seemed like a good update to make, but feel free to be harsh on it so I can improve it. Thanks!!