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

Consider moving Auto-redirect AMP pages below Automatically send diagnostic reports #22007

Closed
Tonev opened this issue Mar 31, 2022 · 1 comment

Comments

@Tonev
Copy link
Contributor

Tonev commented Mar 31, 2022

Description

I'm aware of the reason why Auto-redirect AMP pages was moved, but placing the setting between two settings that do similar things doesn't look/feel right. I believe placing Auto-redirect AMP pages below Automatically send diagnostic reports will provide better user experience, make more sense, and look better visually.

Steps to Reproduce

  1. Install Brave Beta or Brave Nightly.
  2. Open brave://settings/privacy.
  3. Observe the order of aforementioned settings.

Actual result:

Untitled

Expected result:

I strongly believe the following order would make more sense, look better visually, and provide better user experience:

Automatically send daily usage ping to Brave
Automatically send diagnostic reports
Auto-redirect AMP pages

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.39.18 Chromium: 100.0.4896.60 (Official Build) nightly (64-bit)
Revision 6a5d10861ce8de5fce22564658033b43cb7de047-refs/branch-heads/4896@{#875}
OS Windows 7 Service Pack 1 (Build 7601)

Version/Channel Information:

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

Miscellaneous Information:

@ShivanKaul @petemill (brave/brave-core#12694) @rebron

@Tonev
Copy link
Contributor Author

Tonev commented Apr 21, 2022

Resolved after #22262 was implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants