-
Notifications
You must be signed in to change notification settings - Fork 24
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: stream preview power #1204
Conversation
@kevmo314 Anything else to add to this? |
@@ -11,6 +11,7 @@ dependencies: | |||
flutter: | |||
sdk: flutter | |||
audioplayers: ^6.0.0 | |||
battery_plus: ^5.0.2 |
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.
Hmm I think the idea is interesting but I don't think it's worth further increasing our bundle size for this. Right now the prompt shows after a timer which wouldn't have any dependency impact.
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.
This though would be shared anyway if #1202 would still be implemented. Same thoughts on that one as well?
The intent of this is if the battery falls below 20% not charging and not in battery saving mode show the prompt that stream preview takes up lots of battery, then if the battery falls below 5% disable the stream preview if open