-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(device_info_plus)!: Remove Display Metrics from Android Device Info #2731
Conversation
@ChristianEdwardPadilla fyi, please comment on this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good that you mentioned about the DisplayFeature API as I wanted to share it as well :)
I am Ok with dropping this data from the plugin.
LGTM. Thanks the quick turnaround on all this! |
Hi, thanks for the work. Personally I used the device metric to obtain Android device physical info as below before the change:
This was in hope to accurately replicate physical distance on screen using pixel counts. Now that the plugin no longer support the features, what is the alternative way to obtain the number of pixels in both height and width on a screen, and the actual length and width of the device in real world? Thank you. |
There are packages that implement this functionality, also support all platforms and not just Android. e.g. https://github.com/benthillerkus/millimeters |
Description
Following the discussion in #2688 (comment)
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).