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

Brave shields popup can be opened in a new tab #3271

Closed
AnthonyLenglet opened this issue Feb 8, 2019 · 7 comments
Closed

Brave shields popup can be opened in a new tab #3271

AnthonyLenglet opened this issue Feb 8, 2019 · 7 comments
Labels
bug feature/shields The overall Shields feature in Brave. good first issue priority/P4 Planned work. We expect to get to it "soon". QA/Yes release-notes/include

Comments

@AnthonyLenglet
Copy link

Description

The brave shields popup can be opened in a new tab, which shouldn't be possible

Steps to Reproduce

  1. Open the brave shield popup
  2. Middle click the "global shield defaults" link in the popup

Actual result:

The popup is now opened in its own tab
image

Expected result:

Middle click shouldn't do anything

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.59.35 Chromium: 72.0.3626.81 (Official Build) (64-bit)
Revision ac8b982e05014492d1bd7d317628a4f22a97ffa0-refs/branch-heads/3626@{#796}
OS Windows

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? No
  • Is the issue reproducible on the latest version of Chrome? No
@srirambv
Copy link
Contributor

srirambv commented Feb 8, 2019

Nice catch 👍

@tildelowengrimm tildelowengrimm added the priority/P4 Planned work. We expect to get to it "soon". label Feb 12, 2019
@arsalankhalid
Copy link

Can I grab this?

@arsalankhalid
Copy link

@AnthonyLenglet are you performing the middle click through the track pad or mouse? Does it matter?

@AnthonyLenglet
Copy link
Author

I'm doing it through a mouse, don't actually know if it should matter though (my guess would be that it doesn't, because its the shortcut to open a link in a new tab)

@bsclifton
Copy link
Member

@arsalankhalid go for it 😄 Let us know if you need help

@meerasndr
Copy link

This seems solved?

@bsclifton
Copy link
Member

Thanks for raising, @meerasndr - you're right, looks like it's resolved 😄 Closing

@bbondy bbondy added this to the Closed / Invalid milestone Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/shields The overall Shields feature in Brave. good first issue priority/P4 Planned work. We expect to get to it "soon". QA/Yes release-notes/include
Projects
None yet
Development

No branches or pull requests

7 participants