Skip to content

feat: Introduces IconThemeDataMix#766

Closed
tilucasoli wants to merge 3 commits intomainfrom
feat/create-icon-theme-data
Closed

feat: Introduces IconThemeDataMix#766
tilucasoli wants to merge 3 commits intomainfrom
feat/create-icon-theme-data

Conversation

@tilucasoli
Copy link
Copy Markdown
Collaborator

Description

Introduces IconThemeDataMix as a mix representation of IconThemeData and updates IconThemeModifierMix to use it for configuration and merging. Refactors related utility, config, and style mixin code to support the new structure. Updates tests to reflect the new API and property access patterns.

Review Checklist

  • Testing: Have you tested your changes, including unit tests and integration tests for affected code?
  • Breaking Changes: Does this change introduce breaking changes affecting existing code or users?
  • Documentation Updates: Are all relevant documentation files (e.g. README, API docs) updated to reflect the changes in this PR?
  • Website Updates: Is the website containing the updates you make on documentation?

Additional Information (optional)

Is there any additional context or documentation that might be helpful for reviewers?

Introduces IconThemeDataMix as a mix representation of IconThemeData and updates IconThemeModifierMix to use it for configuration and merging. Refactors related utility, config, and style mixin code to support the new structure. Updates tests to reflect the new API and property access patterns.
@docs-page
Copy link
Copy Markdown

docs-page Bot commented Oct 1, 2025

To view this pull requests documentation preview, visit the following URL:

docs.page/btwld/mix~766

Documentation is deployed and generated using docs.page.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Oct 1, 2025

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

Project Deployment Preview Comments Updated (UTC)
mix-docs Ready Ready Preview Comment Oct 1, 2025 10:53pm

Comment thread packages/mix/lib/src/properties/painting/icon_theme_mix.dart Outdated
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
@leoafarias
Copy link
Copy Markdown
Collaborator

Closing — the widget modifier API was significantly refactored in #775 (Dart 3.10 update). This 5-month-old PR has likely diverged too far. If IconThemeDataMix is still desired, a fresh implementation against the current modifier architecture would be cleaner.

@leoafarias leoafarias closed this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants