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

Opening external links in the same tab groups stopped working #32868

Closed
samartnik opened this issue Sep 8, 2023 · 3 comments
Closed

Opening external links in the same tab groups stopped working #32868

samartnik opened this issue Sep 8, 2023 · 3 comments
Assignees
Labels
OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/include

Comments

@samartnik
Copy link
Contributor

samartnik commented Sep 8, 2023

Description

Opening external links in the same tab groups stopped working.

Steps to reproduce

  1. Make sure Only open links in current tab group option is on.
  2. Try to open external link.
  3. It opens in a different tab group (see example https://bravesoftware.slack.com/archives/C0816QH3R/p1694170188294899)

Actual result

Link opens in new tab group.

Expected result

Link opens in the same tab group.

Issue reproduces how often

Always for me, but other devs reported it to be working on their devices.

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? yes
  • Can you reproduce this issue with the current Play Store Beta version? no, it is fixed in cr117
  • Can you reproduce this issue with the current Play Store Nightly version? no, it is fixed in cr117

Device details

  • Install type (ARM, x86):
  • Device type (Phone, Tablet, Phablet):
  • Android version:

Brave version

1.57.x and 1.58.x

Website problems only

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Additional information

@kjozwiak
Copy link
Member

Closing as the above has been fixed via brave/brave-core@145681c as per brave/brave-core#20069 (comment) & brave/brave-core#20070 (comment).

@kjozwiak
Copy link
Member

As per brave/brave-core#20069 (review), the above can be verified with 1.58.120 as per the conversation via https://bravesoftware.slack.com/archives/CHGKGMHDJ/p1694459339315069.

@Uni-verse Uni-verse added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 12, 2023
@Uni-verse
Copy link
Contributor

Uni-verse commented Sep 12, 2023

Verified on Samsung Galaxy S21 using version:

Brave	1.58.120 Chromium: 117.0.5938.48 (Official Build) (32-bit) 
Revision	8b2e2785c90c076fc1413acb2c003e603eac02d3
OS	Android 13; Build/TP1A.220624.014; 33; REL

Test Scenario:

  1. New profile
  2. Confirm tab groups are enabled in appearance settings
  3. Navigate to webpage and open in new tab group using context menu
  4. Repeat again, confirm its opened in same tab group
  5. Confirm using open in new tab will still open in another tab
newtabgroup1.mp4

@Uni-verse Uni-verse added QA Pass - Android ARM and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/include
Projects
None yet
Development

No branches or pull requests

3 participants