-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Comments
In addition to this issue, I found out that |
👋 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 👍 ). |
the above was copied from Figma
i'll be addressing both issues here: brave/brave-core#12523 |
Verified
Used the original steps to reproduce from #21583 (comment). Confirmed @Tonev mind filing separately for the |
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.
|
Verified
|
@Tonev Figma designs (Latest Rev) needs an update as well FYI |
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! |
Description
Just a typo and inconsistency.
Steps to Reproduce
Brave Nightly
.brave://flags/#brave-shields-v2
->
Relaunch
.brave.com
.Advanced Controls
.Actual result:
Expected result:
Note: this may reduce Brave privacy protections.
->
Note: this may reduce Brave's privacy protections.
Filter Lists
andGlobal defaults
->
(Filter lists
andGlobal defaults
) or (Filter Lists
andGlobal Defaults
) so there is consistency.Reproduces how often:
100%
Brave version (brave://version info)
The text was updated successfully, but these errors were encountered: