Skip to content
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

Fixed video FPS hardcoded to 30 on iOs #705

Merged
merged 2 commits into from
Sep 15, 2021

Conversation

antonkrasov
Copy link
Contributor

First of all thanks for the awesome library!

So one of the app users I'm working on complained about FPS on iOs, after some investigation found that it's just hardcoded to 30 FPS, but he sees the difference between 30 and 60 FPS with some videos.

Later on, found this PR - https://github.com/flutter/plugins/pull/3719/files and used it for this PR.

Thanks!

@jhomlala jhomlala changed the base branch from master to feature/august_changes_2 September 15, 2021 16:19
@jhomlala
Copy link
Owner

LGTM, thanks!

@jhomlala jhomlala merged commit 04f4530 into jhomlala:feature/august_changes_2 Sep 15, 2021
jhomlala added a commit that referenced this pull request Sep 20, 2021
* Added expandToFill parameter in BetterPlayerConfiguration

* Updated changelog

* * Added `BetterPlayerControlsConfiguration.theme` factory for `BetterPlayerControlsConfiguration`.

* Added null checks in seek commands in BetterPlayerControlsState.

* Fixed issue with live stream where player controls were always visible.

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Added tests to CI

* Update ci.yml

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Updated tests, general refactor

* Updated cupertino theme

* Fixed iOS seek issue

* Fix iOS caching + add iOS feature: preCaching (#670)

* Fix iOS caching + add iOS feature: preCaching

* Added required changes

* Fix compile issues

* Added stop pre cache iOS implementation

* Updated caching implementation

* Updated caching implementation

* Updated caching implementation

* Updated caching implementation

* Updated documentation

* Fixed video FPS hardcoded to 30 on iOs (#705)

Co-authored-by: Jakub <jhomlala@gmail.com>

* Updated changelog

* set default subtitle from hls (#688)

* set default subtitle from hls

* remove comments

* General refactor

* Disabled analysis options.

* Flutter 2.5 update

* Flutter 2.5 update

* Disabled lint

* Fixed analyzer

* Updated dependencies

* Updated version

Co-authored-by: themadmrj <themadmrj@users.noreply.github.com>
Co-authored-by: Anton Krasov <anton.krasov@gmail.com>
Co-authored-by: Siloe Bezerra Bispo <siloebb@gmail.com>
geriby23 pushed a commit to threadable/betterplayer that referenced this pull request Jul 16, 2024
* Added expandToFill parameter in BetterPlayerConfiguration

* Updated changelog

* * Added `BetterPlayerControlsConfiguration.theme` factory for `BetterPlayerControlsConfiguration`.

* Added null checks in seek commands in BetterPlayerControlsState.

* Fixed issue with live stream where player controls were always visible.

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Added tests to CI

* Update ci.yml

* Updated tests

* Updated tests

* Updated tests

* Updated tests

* Updated tests, general refactor

* Updated cupertino theme

* Fixed iOS seek issue

* Fix iOS caching + add iOS feature: preCaching (jhomlala#670)

* Fix iOS caching + add iOS feature: preCaching

* Added required changes

* Fix compile issues

* Added stop pre cache iOS implementation

* Updated caching implementation

* Updated caching implementation

* Updated caching implementation

* Updated caching implementation

* Updated documentation

* Fixed video FPS hardcoded to 30 on iOs (jhomlala#705)

Co-authored-by: Jakub <jhomlala@gmail.com>

* Updated changelog

* set default subtitle from hls (jhomlala#688)

* set default subtitle from hls

* remove comments

* General refactor

* Disabled analysis options.

* Flutter 2.5 update

* Flutter 2.5 update

* Disabled lint

* Fixed analyzer

* Updated dependencies

* Updated version

Co-authored-by: themadmrj <themadmrj@users.noreply.github.com>
Co-authored-by: Anton Krasov <anton.krasov@gmail.com>
Co-authored-by: Siloe Bezerra Bispo <siloebb@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants