-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Update settings for Shields, and Privacy and security #36859
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
Comments
@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
|
@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. |
Verification
Case 1:
Auto-redirect AMP pages
Case 2:
Data collection
Safety check section
|
Verification
|
1.65.123 |
1.66.69 (light theme) |
1.66.69 (dark theme) |
---|---|---|
![]() |
![]() |
![]() |
-
Confirmed following settings are moved from
brave://settings/shields
tobrave://settings/privacy
and are shown right belowUse 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) |
---|---|---|
![]() |
![]() |
![]() |
Auto-redirect AMP pages
- opened
brave://settings/privacy
- enabled
Auto-redirect AMP pages
setting as default - loaded https://shivankaul.com/brave/de-amp/test-amp-lightning-empty-attribute.html
- Confirmed
Auto-redirect AMP pages
toggleon/off
work as expected
example | example |
---|---|
![]() |
![]() |
![]() |
![]() |
Case 2: brave://settings/privacy
- Confirmed following settings moved from bottom to the top of the
Privacy and Security
section in thebrave://settings/privacy
- Clear browsing data
- Security
- Site and Shields
1.65.123 |
1.66.69 (light theme |
1.66.69 (dark theme) |
---|---|---|
![]() |
![]() |
![]() |
Data collection
- Confirmed a new section called
Data collection
displayed below theTor windows
section and following settings are shown in the section in thebrave://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) |
---|---|---|
![]() |
![]() |
![]() |
Safety check section
- Confirmed
Safety check
section is no longer shown at the bottom ofbrave://settings/privacy
page
1.65.123 | 1.66.59 |
---|---|
![]() |
![]() |
Verified
|
1.65.123 |
1.66.69 (light theme) |
1.66.69 (dark theme) |
---|---|---|
![]() |
![]() |
![]() |
-
Confirmed following settings are moved from
brave://settings/shields
tobrave://settings/privacy
and are shown right belowUse 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) |
---|---|---|
![]() |
![]() |
![]() |
Auto-redirect AMP pages
- opened
brave://settings/privacy
- enabled
Auto-redirect AMP pages
setting as default - loaded https://shivankaul.com/brave/de-amp/test-amp-lightning-empty-attribute.html
- Confirmed
Auto-redirect AMP pages
toggleon/off
work as expected
example | example |
---|---|
![]() |
![]() |
![]() |
![]() |
Case 2: brave://settings/privacy
- Confirmed following settings moved from bottom to the top of the
Privacy and Security
section in thebrave://settings/privacy
- Clear browsing data
- Security
- Site and Shields
1.65.123 |
1.66.69 (light theme) |
1.66.69 (dark theme) |
---|---|---|
![]() |
![]() |
![]() |
Data collection
- Confirmed a new section called
Data collection
displayed below theTor windows
section and following settings are shown in the section in thebrave://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) |
---|---|---|
![]() |
![]() |
![]() |
Safety check section
- Confirmed
Safety check
section is no longer shown at the bottom ofbrave://settings/privacy
page
1.65.123 | 1.66.59 |
---|---|
![]() |
![]() |
Description
Follow-up to #36363
In brave://settings/shields:
Show the number of blocked items on the Shields icon
setting aboveTrackers & ads blocking
Auto-redirect AMP pages
,Auto-redirect tracking URLs
, andPrevent sites from fingerprinting me based on my language
to brave://settings/privacy. Right belowUse Google services for push messaging
In brave://settings/privacy:
Clear browsing data
,Security
, andSite and Shields settings
to the top of thePrivacy
section.Data collection
below theTor windows
section and move the Allow p3a, Allow ping, and Allow diagnostic reports into that section.Safety check
section.Design
Reproduces how often:
n/a
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc: @ShivanKaul, @fmarier, @diracdeltas
The text was updated successfully, but these errors were encountered: