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

Shields v2: Some texts need a change #21583

Closed
Tonev opened this issue Mar 10, 2022 · 8 comments · Fixed by brave/brave-core#12523
Closed

Shields v2: Some texts need a change #21583

Tonev opened this issue Mar 10, 2022 · 8 comments · Fixed by brave/brave-core#12523
Assignees
Labels

Comments

@Tonev
Copy link
Contributor

Tonev commented Mar 10, 2022

Description

Just a typo and inconsistency.

Steps to Reproduce

  1. Install Brave Nightly.
  2. Enable brave://flags/#brave-shields-v2 -> Relaunch.
  3. Open a random website, for example brave.com.
  4. Open the Shields panel via the Shields icon.
  5. Click on Advanced Controls.

Actual result:

muOaXS6aXe

Expected result:

  1. Note: this may reduce Brave privacy protections. -> Note: this may reduce Brave's privacy protections.
  2. Filter Lists and Global defaults -> (Filter lists and Global defaults) or (Filter Lists and Global Defaults) so there is consistency.

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.38.32 Chromium: 99.0.4844.51 (Official Build) nightly (64-bit)
Revision d537ec02474b5afe23684e7963d538896c63ac77-refs/branch-heads/4844@{#875}
OS Windows 7 Service Pack 1 (Build 7601)
@Tonev
Copy link
Contributor Author

Tonev commented Mar 10, 2022

In addition to this issue, I found out that Learn more, Filter Lists, and Global defaults options don't do anything. Please, let me know if you would like me to open another ticket for that, or is there one already? Thanks.

@stephendonner @nullhook @kjozwiak

@stephendonner
Copy link

In addition to this issue, I found out that Learn more, Filter Lists, and Global defaults options don't do anything. Please, let me know if you would like me to open another ticket for that, or is there one already? Thanks.

@stephendonner @nullhook @kjozwiak

👋 I think that's expected for now given it's still behind a feature flag; that said, perhaps @nullhook is ready to hit this heavily with bugs (I think so, but will let him confirm 👍 ).

@stephendonner stephendonner added feature/shields/panel Front-end design and functionality of the Shields panel. needs-text-change This change requires some careful wording. bug QA/Yes labels Mar 10, 2022
@nullhook
Copy link

nullhook commented Mar 11, 2022

Note: this may reduce Brave privacy protections.

the above was copied from Figma

In addition to this issue, I found out that Learn more, Filter Lists, and Global defaults options don't do anything. Please, let me know if you would like me to open another ticket for that, or is there one already? Thanks.

i'll be addressing both issues here: brave/brave-core#12523

@stephendonner
Copy link

Verified PASSED using

Brave 1.39.10 Chromium: 100.0.4896.46 (Official Build) nightly (x86_64)
Revision 5ca33821b2211805855c77d334353d27c616a7ca-refs/branch-heads/4896@{#584}
OS macOS Version 11.6.5 (Build 20G527)

Used the original steps to reproduce from #21583 (comment).

Confirmed Brave's privacy protections

Screen Shot 2022-03-28 at 4 07 44 PM

@Tonev mind filing separately for the Filter Lists -> Filter lists issue?

@Tonev
Copy link
Contributor Author

Tonev commented Mar 31, 2022

Sorry for getting late to you, @stephendonner, somehow missed your email. There was no need for a new ticket as @nullhook addressed the matter in another pull request (brave/brave-core#12775), just at a later point so your testing wasn't able to catch it.

Brave 1.39.22 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)

image

@MadhaviSeelam
Copy link

MadhaviSeelam commented Apr 1, 2022

Verified Passed using

Brave 1.38.77 Chromium: 100.0.4896.60 (Official Build) beta (64-bit)
Revision 6a5d10861ce8de5fce22564658033b43cb7de047-refs/branch-heads/4896@{#875}
OS Windows 11 Version 21H2 (Build 22000.593)
  • Install 1.38.77
  • Launch Brave
  • Enable brave://flags/#brave-shields-v2 -> Relaunch.
  • Visited brave.com.
  • Open the Shields panel via the Shields icon.
  • Verified Shields V2 text for Brave's privacy protections

Shields v2-text change

@MadhaviSeelam
Copy link

MadhaviSeelam commented Apr 1, 2022

@Tonev Figma designs (Latest Rev) needs an update as well FYI

image

@Tonev
Copy link
Contributor Author

Tonev commented Apr 2, 2022

@MadhaviSeelam

Thanks for telling me, but I'm afraid I won't be able to do anything about it. If folks keep updating the designs, and the designs don't future the changes from this ticket, I'm afraid it would be a lost cause - someone will update through a PR, and then someone else's PR that updates the designs will erase the previous PR's changes.

CC: @rebron @stephendonner @aguscruiz @nullhook for thoughts on the matter, please!

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

Successfully merging a pull request may close this issue.

5 participants