feat(device_info_plus)!: migrate to package:web #2624
Merged
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
dart:html
by the equivalent with thepackage:web
.>=0.5.0 <=0.6.0
to be compatible with any upcomingweb
release.3.3.0
in package, as it is the min version ofweb
0.5
. This is required sinceNavigator
is only available from version0.5
, so it is not possible to be compatible with older versions ofweb
.deviceMemory
seems to be missing in the current web implementation. The corresponding ticket has been created in thepackage:web
repo: Navigator.deviceMemory is missing dart-lang/web#192 This property is nullable (int?
) so it is not a blocker, however, users relying on this property will be affected. A solution is possible by creating our owndeviceMemory
extension.Related Issues
Checklist
CHANGELOG.md
nor the plugin version inpubspec.yaml
files.flutter analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?
!
in the title as explained in Conventional Commits).