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

Update settings for Shields, and Privacy and security #36859

Closed
5 tasks done
rebron opened this issue Mar 15, 2024 · 5 comments · Fixed by brave/brave-core#22662
Closed
5 tasks done

Update settings for Shields, and Privacy and security #36859

rebron opened this issue Mar 15, 2024 · 5 comments · Fixed by brave/brave-core#22662
Assignees
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/settings OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-All-Platforms QA/Yes release-notes/include

Comments

@rebron
Copy link
Collaborator

rebron commented Mar 15, 2024

Description

Follow-up to #36363

In brave://settings/shields:

  • Move Show the number of blocked items on the Shields icon setting above Trackers & ads blocking
  • Move Auto-redirect AMP pages, Auto-redirect tracking URLs, and Prevent sites from fingerprinting me based on my language to brave://settings/privacy. Right below Use Google services for push messaging

In brave://settings/privacy:

  • Move Clear browsing data, Security, and Site and Shields settings to the top of the Privacy section.
  • Create a new section called Data collection below the Tor windows section and move the Allow p3a, Allow ping, and Allow diagnostic reports into that section.
  • Remove Safety check section.

Design

Screenshot 2024-03-19 at 5 09 25 PM Screenshot 2024-03-19 at 5 09 38 PM

Reproduces how often:

n/a

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

cc: @ShivanKaul, @fmarier, @diracdeltas

@ShivanKaul
Copy link
Collaborator

@emerick I don't know if you'd gotten to the 3rd point in the issue for changes in brave://settings/shields, but just wanted to call out that I added the requirement for the sub-label for Block cookies.

Move Third-party cookies from brave://settings/privacy to brave://settings/shields, fully replacing Block cookies and the drop down menu in favor of this link to the sub-menu. Keep the top-level label Block cookies and the sub-label from Third-party cookies (reflecting current status).

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 18, 2024
@rebron rebron added this to In progress in Settings Mar 19, 2024
@emerick
Copy link
Contributor

emerick commented Mar 20, 2024

@rebron This is just about done (all checked items are finished), except I haven't removed the original "Block cookies" dropdown as you mentioned that it still needs to be supported somewhere. Let me know if you have any thoughts as far as where it should go. At the moment, I just have it in brave://settings/shields right above the "Block cookies >" entry.

@emerick emerick added this to the 1.66.x - Nightly milestone Mar 22, 2024
@rebron rebron moved this from In progress to Completed in Settings Mar 26, 2024
@MadhaviSeelam MadhaviSeelam added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Apr 4, 2024
@MadhaviSeelam
Copy link

MadhaviSeelam commented Apr 4, 2024

Verification PASSED using

Brave | 1.66.59 Chromium: 123.0.6312.105 (Official Build) nightly (64-bit)
-- | --
Revision | 2714c7ac837e7b7693b1b56a82effdf57e04fe09
OS | Windows 11 Version 23H2 (Build 22631.3374)
  1. Installed 1.66.59
  2. launched Brave
  3. opened brave://settings/shields
  4. opened brave://settings/privacy

Case 1: brave://settings/shields

  • Confirmed Show the number of blocked items on the Shields icon setting is shown above Trackers & ads blocking
  • Confirmed the toggle off/on to show blocked items work as expected
1.64.116 1.66.69(light theme) 1.66.69 (dark theme)
image image image
  • Confirmed following settings are moved from brave://settings/shields to brave://settings/privacy and are shown right below Use Google services for push messaging

    • Auto-redirect AMP pages
    • Auto-redirect tracking URLs
    • Prevent sites from fingerprinting me based on my language
  • Confirmed Learn more links work as expected
    Note: Support pages need an update

1.64.116 1.66.69(light theme) 1.66.69 (dark theme)
image image image

Auto-redirect AMP pages

example example
image image
image image

Case 2: brave://settings/privacy

  • Confirmed following settings moved from bottom to the top of the Privacy and Security section in the brave://settings/privacy
    • Clear browsing data
    • Security
    • Site and Shields
1.64.116 1.66.69(light theme 1.66.69 (dark theme)
image image image

Data collection

  • Confirmed a new section called Data collection displayed below the Tor windows section and following settings are shown in the section in the brave://settings/privacy
    • Allow privacy-preserving product analytics (P3A)
    • Automatically send daily usage ping to Brave
    • Automatically send diagnostic reports
  • Confirmed toggle on/off work as expected for above settings
1.64.116 1.66.69(light theme) 1.66.69 (dark theme)
image image image

Safety check section

  • Confirmed Safety check section is no longer shown at the bottom of brave://settings/privacy page
1.64.116 1.66.59
image image

@MadhaviSeelam MadhaviSeelam added QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 4, 2024
@stephendonner
Copy link

stephendonner commented Apr 29, 2024

Verification PASSED using

Brave | 1.66.94 Chromium: 124.0.6367.91 (Official Build) beta (x86_64)
-- | --
Revision | 1574096f9d2bcb69c3654cb4437217121ee9600b
OS | macOS Version 11.7.10 (Build 20G1427)

Steps:

  1. Installed 1.66.94
  2. launched Brave
  3. opened brave://settings/shields
  4. opened brave://settings/privacy

Case 1: brave://settings/shields

  • Confirmed Show the number of blocked items on the Shields icon setting is shown above Trackers & ads blocking
  • Confirmed the toggle off/on to show blocked items work as expected
1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 11 41 00 AM Screen Shot 2024-04-29 at 11 37 58 AM Screen Shot 2024-04-29 at 11 38 36 AM
  • Confirmed following settings are moved from brave://settings/shields to brave://settings/privacy and are shown right below Use Google services for push messaging

    • Auto-redirect AMP pages
    • Auto-redirect tracking URLs
    • Prevent sites from fingerprinting me based on my language
  • Confirmed Learn more links work as expected
    Note: Support pages need an update

1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 11 49 28 AM Screen Shot 2024-04-29 at 11 51 24 AM Screen Shot 2024-04-29 at 11 50 54 AM

Auto-redirect AMP pages

example example
Screen Shot 2024-04-29 at 11 57 10 AM Screen Shot 2024-04-29 at 11 58 21 AM
Screen Shot 2024-04-29 at 11 57 05 AM Screen Shot 2024-04-29 at 11 58 27 AM

Case 2: brave://settings/privacy

  • Confirmed following settings moved from bottom to the top of the Privacy and Security section in the brave://settings/privacy
    • Clear browsing data
    • Security
    • Site and Shields
1.65.123 1.66.69(light theme 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 12 04 23 PM Screen Shot 2024-04-29 at 12 05 47 PM Screen Shot 2024-04-29 at 12 06 04 PM

Data collection

  • Confirmed a new section called Data collection displayed below the Tor windows section and following settings are shown in the section in the brave://settings/privacy
    • Allow privacy-preserving product analytics (P3A)
    • Automatically send daily usage ping to Brave
    • Automatically send diagnostic reports
  • Confirmed toggle on/off work as expected for above settings
1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 12 07 11 PM Screen Shot 2024-04-29 at 12 08 20 PM Screen Shot 2024-04-29 at 12 08 00 PM

Safety check section

  • Confirmed Safety check section is no longer shown at the bottom of brave://settings/privacy page
1.65.123 1.66.59
Screen Shot 2024-04-29 at 12 11 32 PM Screen Shot 2024-04-29 at 12 12 26 PM

@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 29, 2024
@stephendonner
Copy link

stephendonner commented Apr 30, 2024

Verified PASSED using

Brave | 1.66.94 Chromium: 124.0.6367.91 (Official Build) beta (64-bit) 
-- | --
Revision | 1574096f9d2bcb69c3654cb4437217121ee9600b
OS | Linux

Steps:

  1. Installed 1.66.94
  2. launched Brave
  3. opened brave://settings/shields
  4. opened brave://settings/privacy

Case 1: brave://settings/shields

  • Confirmed Show the number of blocked items on the Shields icon setting is shown above Trackers & ads blocking
  • Confirmed the toggle off/on to show blocked items work as expected
1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 7 01 57 PM Screen Shot 2024-04-29 at 6 43 23 PM Screen Shot 2024-04-29 at 6 47 27 PM
  • Confirmed following settings are moved from brave://settings/shields to brave://settings/privacy and are shown right below Use Google services for push messaging

    • Auto-redirect AMP pages
    • Auto-redirect tracking URLs
    • Prevent sites from fingerprinting me based on my language
  • Confirmed Learn more links work as expected
    Note: Support pages need an update

1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 7 04 44 PM Screen Shot 2024-04-29 at 7 06 06 PM Screen Shot 2024-04-29 at 7 05 47 PM

Auto-redirect AMP pages

example example
Screen Shot 2024-04-29 at 7 13 21 PM Screen Shot 2024-04-29 at 7 13 01 PM
Screen Shot 2024-04-29 at 7 13 24 PM Screen Shot 2024-04-29 at 7 14 36 PM

Case 2: brave://settings/privacy

  • Confirmed following settings moved from bottom to the top of the Privacy and Security section in the brave://settings/privacy
    • Clear browsing data
    • Security
    • Site and Shields
1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 7 23 49 PM Screen Shot 2024-04-29 at 7 22 48 PM Screen Shot 2024-04-29 at 7 23 08 PM

Data collection

  • Confirmed a new section called Data collection displayed below the Tor windows section and following settings are shown in the section in the brave://settings/privacy
    • Allow privacy-preserving product analytics (P3A)
    • Automatically send daily usage ping to Brave
    • Automatically send diagnostic reports
  • Confirmed toggle on/off work as expected for above settings
1.65.123 1.66.69(light theme) 1.66.69 (dark theme)
Screen Shot 2024-04-29 at 7 26 48 PM Screen Shot 2024-04-29 at 7 28 32 PM Screen Shot 2024-04-29 at 7 28 09 PM

Safety check section

  • Confirmed Safety check section is no longer shown at the bottom of brave://settings/privacy page
1.65.123 1.66.59
Screen Shot 2024-04-29 at 7 31 40 PM Screen Shot 2024-04-29 at 7 31 23 PM

@stephendonner stephendonner added QA Pass-Linux and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/settings OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-All-Platforms QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants