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

advanced options toggle should be a global option instead of per site #471

Closed
cezaraugusto opened this issue Jul 2, 2018 · 3 comments · Fixed by brave/brave-extension#78
Assignees
Labels
feature/shields The overall Shields feature in Brave. QA/No release-notes/exclude

Comments

@cezaraugusto
Copy link
Contributor

to make this shieldsPanelTypes.CONTROLS_TOGGLED should not set the state as a per-site setting but rather a global setting.

@cezaraugusto cezaraugusto added the feature/shields The overall Shields feature in Brave. label Jul 2, 2018
@cezaraugusto cezaraugusto added this to the Backlog milestone Jul 2, 2018
@cezaraugusto cezaraugusto self-assigned this Jul 2, 2018
cezaraugusto added a commit to brave/brave-extension that referenced this issue Oct 11, 2018
cezaraugusto added a commit to brave/brave-extension that referenced this issue Oct 11, 2018
cezaraugusto added a commit to brave/brave-extension that referenced this issue Oct 11, 2018
@bsclifton
Copy link
Member

@cezaraugusto do you have more information about this? I'm not sure what you mean in your original post

When visiting a site with the v1 shields, the advanced settings were being saved on a per-site basis. When we re-work the shields with #1196, it should still work like that right?

@cezaraugusto
Copy link
Contributor Author

ya I meant that if you toggle the advanced options visibility in one site the option should be toggled across all sites but this will be fixed by the new version of shields as the option will be removed

@bsclifton
Copy link
Member

@cezaraugusto I marked as QA/No; if any manual QA is required (I can't think of any?), please update original issue to include some test steps (or what to look for) and mark it as QA/Yes. Thanks 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/shields The overall Shields feature in Brave. QA/No release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants