Skip to content

docs(mix): consolidate chained widget modifiers - #57

Merged
tilucasoli merged 1 commit into
mainfrom
docs/consolidate-wrap-examples
Oct 6, 2026
Merged

tilucasoli merged 1 commit into
mainfrom
docs/consolidate-wrap-examples

Conversation

@leoafarias

@leoafarias leoafarias commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

The modifier guide repeats .wrap() where one chained configuration expresses the same style. Consolidate the card example and both default-order examples, preserving the comparison that demonstrates order independence for distinct modifier types.

Consolidate the custom-order example and correct its comments: the first resolved modifier is the outermost wrapper. No runtime API changes.

Validation:

  • 46 site tests passed.
  • Flutter preview build and manifest validation passed (74 entries).
  • Production website build and Pagefind indexing passed (existing lint/Nextra warnings).
  • The companion Mix PR refactor(mix): consolidate example widget modifiers mix#1090 verifies style equality and default/custom wrapper rendering; 30 focused modifier tests passed.
  • git diff --check passed.

GitHub CI: all checks passed for 1f7ee27890daf9dee2923dcbc7a50ed0e820c19c, including preview artifacts + website, CodeQL, and Vercel deployment.

@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
mix-docs Ready Ready Preview Oct 6, 2026 3:07am UTC

Request Review

@tilucasoli tilucasoli left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@tilucasoli
tilucasoli merged commit 4cb58bf into main Oct 6, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
Preview – mix-docs — 1f7ee278 Deployed Oct 6, 2026 by vercel[bot]
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.

2 participants