Migrate to the standalone material_ui / cupertino_ui packages - #2765
Open
putnokiabel wants to merge 1 commit into
Open
Migrate to the standalone material_ui / cupertino_ui packages#2765putnokiabel wants to merge 1 commit into
putnokiabel wants to merge 1 commit into
Conversation
Apply `dart fix --apply --code=migrate_design_widgets` to move imports from package:flutter/material.dart and package:flutter/cupertino.dart to the standalone package:material_ui/material_ui.dart and package:cupertino_ui/cupertino_ui.dart packages, per the Flutter breaking change: https://docs.flutter.dev/release/breaking-changes/material-ui-and-cupertino-ui - Rewrite imports across flutter_quill, flutter_quill_extensions, flutter_quill_test and example (migrate_design_widgets + directives_ordering). - Add material_ui / cupertino_ui dependencies to each affected package. - example: hide GlobalMaterialLocalizations from the material_ui import to resolve the ambiguity with flutter_localizations. - flutter_quill_extensions: add pubspec_overrides.yaml so it builds against the in-repo (migrated) flutter_quill instead of the published release that still imports package:flutter/material.dart. The override is ignored by consumers and excluded from `pub publish`. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Description
Migrates the packages off
package:flutter/material.dart/package:flutter/cupertino.dartand onto the new standalonematerial_ui/cupertino_uipackages, following the Flutter breaking change:https://docs.flutter.dev/release/breaking-changes/material-ui-and-cupertino-ui
How it was done
Following the documented procedure:
Applied across all packages:
flutter_quill,flutter_quill_extensions,flutter_quill_testandexample.Changes
package:flutter/material.dart→package:material_ui/material_ui.dartandpackage:flutter/cupertino.dart→package:cupertino_ui/cupertino_ui.dart.material_ui: ^1.2.0(all packages) andcupertino_ui: ^1.0.2(packages that use Cupertino widgets).example/lib/main.dart:hide GlobalMaterialLocalizationson thematerial_uiimport — it now also exports that symbol, which was ambiguous againstflutter_localizations.flutter_quill_extensions/pubspec_overrides.yaml: path override so the package builds against the in-repo (migrated)flutter_quillrather than the published release (which still importspackage:flutter/material.dart, breaking theThemeData.isCupertinoextension). The override is ignored by consumers and excluded frompub publish, so it only affects local/CI resolution during the transition.Verification
dart analyze— no issues in all four packages (lib + tests).dart format --set-exit-if-changed .— clean.Note for maintainers
CI (
.github/workflows/checks.yml) pinsflutter-version: 3.44.1. Thematerial_ui/cupertino_uipackages require a newer Flutter (verified locally on 3.47.2), so the pinned CI version will need bumping alongside this migration.