Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Advanced block autoplay settings #8017

Closed
vesta0 opened this issue Jan 31, 2020 · 16 comments · Fixed by #9372
Closed

Advanced block autoplay settings #8017

vesta0 opened this issue Jan 31, 2020 · 16 comments · Fixed by #9372
Assignees

Comments

@vesta0
Copy link
Collaborator

vesta0 commented Jan 31, 2020

User Story

As a user, I want to be able to block Audio only or choose how autoplay is blocked over wifi, so I can control my experience to match my needs.

Dependencies

GV: https://bugzilla.mozilla.org/show_bug.cgi?id=1593843#c27

See #255 for original issue

See mocks

Acceptance Criteria

-I can choose to only block audio. When I block audio only, only audio is blocked
-I can choose to allow audio and video over wifi only. When I am not on wifi, audio and video are blocked, and when I am on wifi audio and video auto play.

image

┆Issue is synchronized with this Jira Task

@github-actions github-actions bot added the needs:triage Issue needs triage label Jan 31, 2020
@vesta0 vesta0 mentioned this issue Feb 4, 2020
2 tasks
@vesta0 vesta0 added Feature:Battery 🙅 waiting Issues that are blocked or has dependencies that are not ready labels Feb 4, 2020
@severinrudie severinrudie self-assigned this Feb 29, 2020
severinrudie added a commit to severinrudie/fenix that referenced this issue Mar 3, 2020
severinrudie added a commit to severinrudie/fenix that referenced this issue Mar 3, 2020
severinrudie pushed a commit that referenced this issue Mar 3, 2020
@severinrudie severinrudie removed the needs:triage Issue needs triage label Mar 4, 2020
@severinrudie
Copy link
Contributor

It looks like newer versions of Android encourage us to check "is this an unmetered connection?" instead of "are we on WIFI?" @vesta0 should we stick with "allow on WIFI" or change to "allow on unmetered"?

@vesta0
Copy link
Collaborator Author

vesta0 commented Mar 10, 2020

Good point @baron-severin Let's see what @betsymi thinks!

severinrudie pushed a commit that referenced this issue Mar 10, 2020
severinrudie added a commit to severinrudie/fenix that referenced this issue Mar 11, 2020
@liuche liuche mentioned this issue Mar 12, 2020
32 tasks
@sv-ohorvath
Copy link
Contributor

sv-ohorvath commented Mar 12, 2020

@vesta0 @baron-severin, @liuche included this in the 4.1 release notes, but I don't see any changes on Beta 4.1 or Nightly and the PR looks like is still open. Should there be anything to verify on this issue for now?

@betsymi
Copy link

betsymi commented Mar 12, 2020

@vesta0 @baron-severin @liuche included this in the 4.1 release notes, but I don't see any changes on Beta 4.1 or Nightly and the PR looks like is still open. Should there be anything to verify on this issue for now?

Am still working on unpacking this. For a future state, I don't think this is a simple matter of changing the strings we have. May need to make adjustments in the design to provide more clarity for users, particularly regarding Allow audio and video on Wi-Fi only. Does this means that audio and video will be blocked on cellular data?

@liuche
Copy link
Contributor

liuche commented Mar 12, 2020

@sv-ohorvath that was my mistake! I've removed it from the release notes.

@sv-sdeiac sv-sdeiac removed the eng:qa:needed QA Needed label Mar 23, 2020
@severinrudie
Copy link
Contributor

Additionally. I tested the "Block audio and video on cellular data only" option and it seems that this option is not working. The autoplay is still playing even when I am on the mobile data.

I'm on PTO this week, @Amejia481 would you mind taking a quick look? This is the BZ ticket, this is the code that I thought blocked on cellular data.

The easiest approach here might just be to reenable our wifi logic. That should be super easy. Search in the project for 1621825 and you'll find comments about how to hook it back up.

@Amejia481
Copy link
Contributor

No problem I can take a look!

@Amejia481 Amejia481 self-assigned this Mar 23, 2020
@psymoon psymoon added the 🐞 bug Crashes, Something isn't working, .. label Mar 23, 2020
@liuche liuche mentioned this issue Mar 24, 2020
32 tasks
@sv-ohorvath
Copy link
Contributor

sv-ohorvath commented Mar 24, 2020

@liuche FYI, this has landed on Beta 4.2 with a defect, see #8017 (comment). And there is #5636 as well.

Amejia481 added a commit to Amejia481/fenix that referenced this issue Mar 24, 2020
@Amejia481
Copy link
Contributor

Hi, I can confirm that this issue is fixed using the latest Firefox Preview Nightly 3/23 #20830607.
Also, thanks @baron-severin for your information.

Devices:

* Google Pixel 3 (Android 10);

* Samsung Galaxy Tab S6 (Android 9);

* LG g7 fit (Android 8.1).

Additionally. I tested the "Block audio and video on cellular data only" option and it seems that this option is not working. The autoplay is still playing even when I am on the mobile data.
@baron-severin Can you take a look there and come with more info? I'll remove the qa:needed and wait for your updates, thanks.

This pr #9372 should fix the issue.

@Amejia481 Amejia481 linked a pull request Mar 24, 2020 that will close this issue
4 tasks
@Amejia481 Amejia481 reopened this Mar 28, 2020
@Amejia481 Amejia481 added the eng:qa:needed QA Needed label Mar 28, 2020
@sv-sdeiac
Copy link

Hi, I can confirm that the issue found previously is now fixed using the latest version of Firefox Preview Nightly 3/29 #20891805.

Device: Huawei P40 lite (Android 9).

@sv-sdeiac sv-sdeiac added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Mar 30, 2020
@liuche liuche mentioned this issue Apr 13, 2020
32 tasks
@liuche liuche mentioned this issue Apr 28, 2020
32 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants