-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Hide UI elements related to IdleDetection #18409
Comments
|
Split out Android into a separate issue: #18414 |
Going to change this from |
Verified
|
Brave | 1.31.73 Chromium: 94.0.4606.71 (Official Build) beta (x86_64) |
---|---|
Revision | 1d32b169326531e600d836bd395efc1b53d0f6ef-refs/branch-heads/4606@{#1256} |
OS | macOS Version 11.6 (Build 20G165) |
Steps:
- Start Brave and navigate to any site (e.g.
cnn.com
) - Navigate to
brave://settings
→
"Privacy and security"
→
"Site and Shields Settings"
- Under
"Permissions"
section, click on"Additional permissions"
- Verify that
"Your device use"
item is not present - Scroll back to the top of the page and click on
"View permissions and data stored across sites"
- Pick the site you visited in step 1 and click to expand
- Click on the site
- Verify that
"Your device use"
item is not present - Navigate to
brave://settings/content/idleDetection
- Verify that the page is empty.
example | example | example |
---|---|---|
Verify that the UI is not hidden when the feature is enabled:
- Start Brave from the command line with
--enable-features=IdleDetection
- Repeat steps above, expect the reverse in the Verify steps.
example | example | example |
---|---|---|
Following up from #18269 (comment), Brave currently disables the Idle Detection API, but doesn't remove the UI elements / settings around it, causing confusion.
To prevent confusion, we should just hide the UI elements related to the Idle Detection API
The text was updated successfully, but these errors were encountered: