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

move private window with Tor new tab page support links inline to avoid CAPTCHAs #1647

Open
kjozwiak opened this issue Oct 17, 2018 · 4 comments
Assignees
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. documentation ✍️ feature/tor OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Oct 17, 2018

Description

Support links within the Tor window are being opened in new tabs and not inline as per #1507. Because they're being opened in new tabs, you'll run into a lot of CAPTCHA checks. I've personally one through at least 20 CAPTCHA checks and haven't passed once. Every time I get the ✅, another CAPTCHA is loaded after the page is refreshed.

Steps to Reproduce

  1. launch 0.55.16 Chromium: 70.0.3538.54
  2. open a new Tor window
  3. under brave://newtab, click on the "Learn more" link which will load https://support.brave.com/hc/en-us/articles/360018121491 in a new tab

Actual result:

torissue

Expected result:

Should be opening inline as per #1507 rather than new tabs.

Reproduces how often:

100% reproducible using the above STR.

Brave version (brave://version info)

Went through verification using the following build under macOS 10.13.6 x64 - PASSED

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Mac OS X

Reproducible on current release:

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

I'm not 100% certain as we still haven't received beta & dev builds that have the new brave://newtab changes but I'm assuming it's going to be the same issue across the different channels.

  • Does it reproduce on browser-laptop?

N/A

Additional Information

CCing @rebron

@GeetaSarvadnya
Copy link

Issue is reproducible on Windows.

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Windows

@rebron
Copy link
Collaborator

rebron commented Oct 17, 2018

Didn't die via captcha, was able to get through on first pass, not a great experience to have to go through a captcha for a learn more link. Got a separate issue where article page isn't styled. Probably a good idea to move content inline.
screen shot 2018-10-16 at 11 04 32 pm

@tildelowengrimm tildelowengrimm modified the milestones: 1.0, 0.56.x - Beta Oct 17, 2018
@tildelowengrimm tildelowengrimm changed the title support links under tor window are being opened in new tabs and not inline, causing CAPTCHA death move private window with Tor new tab page support links inline to avoid CAPTCHAs Oct 17, 2018
@tildelowengrimm
Copy link
Contributor

CAPTCHAs are a bad look. Since we know that the links on the Tor new tab page are going to be opened in Tor, let's move them back to an inline overlay (no network) while we work to make sure the knowledge-base is accessible over Tor without any additional work needed from the user.

@rebron rebron removed this from the 0.56.x - Beta milestone Oct 30, 2018
@bbondy bbondy added this to the 1.x Backlog milestone Oct 30, 2018
@tildelowengrimm tildelowengrimm added the priority/P3 The next thing for us to work on. It'll ride the trains. label Oct 30, 2018
@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@rossmoody rossmoody removed their assignment May 24, 2019
@cezaraugusto
Copy link
Contributor

FWIW opening links in the same tab don't run away from CAPTCHA. The best solution is to inline their content in a modal like what Tom said or use some other content block. Removing my assignment for now since this needs product/design spec first.

@cezaraugusto cezaraugusto removed their assignment Jan 17, 2020
@tildelowengrimm tildelowengrimm added the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. documentation ✍️ feature/tor OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes
Projects
None yet
Development

No branches or pull requests

8 participants