Add background tap to pause video feature

[Ortes]

When the video play and the feature enabled one click anywhere on the overlay pause the video

[maherjaafar]

Can you add a video to demonstrate how this feature works?

[Ortes]

Sure when clicking anywhere on the video the video play/pause instead of just making the overlay appear

Screen.Recording.2025-04-01.at.19.33.00.mov

[diegotori]

@Ortes please fix the formatting issues as per the CI results. Thanks.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 22 lines in your changes missing coverage. Please review.

Project coverage is 44.55%. Comparing base (2aca16e) to head (9c375bd).
Report is 5 commits behind head on master.

Files with missing lines	Patch %	Lines
lib/src/material/material_desktop_controls.dart	0.00%	9 Missing ⚠️
lib/src/cupertino/cupertino_controls.dart	0.00%	6 Missing ⚠️
lib/src/material/material_controls.dart	0.00%	6 Missing ⚠️
lib/src/chewie_player.dart	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #875      +/-   ##
==========================================
- Coverage   44.95%   44.55%   -0.41%     
==========================================
  Files          22       22              
  Lines        1546     1560      +14     
==========================================
  Hits          695      695              
- Misses        851      865      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Ortes]

@Ortes please fix the formatting issues as per the CI results. Thanks.

Is this ok ?

[diegotori]

LGTM.

[Ortes]

It might be a good idea to rename the parameter if pauseOnBackgroundTap is unclear

Possible property rename.

[diegotori]

It might be a good idea to rename the parameter if pauseOnBackgroundTap is unclear

Up to you.

[Ortes]

I would rather have this as the default behavior as it is the behavior on Youtube and a lot of other players out here. But I understand if you don't want breaking changes

[diegotori]

I would rather have this as the default behavior as it is the behavior on Youtube and a lot of other players out here. But I understand if you don't want breaking changes

Yeah, it should be well documented for now that this option is available. In a future minor update, we'll turn it on by default.

I guess your task right now is to document the hell out of this for the time being. Thanks.

[diegotori]

LGTM.