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

Tab Groups toggle glitch #16140

Closed
mizzunet opened this issue May 30, 2021 · 2 comments · Fixed by brave/brave-core#8998
Closed

Tab Groups toggle glitch #16140

mizzunet opened this issue May 30, 2021 · 2 comments · Fixed by brave/brave-core#8998
Assignees
Labels
bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA/Yes regression release/blocking release-notes/include

Comments

@mizzunet
Copy link

Description

Tab Groups can't be turned off.

Steps to reproduce

  1. Update to Version 1.25.69
  2. Disable Tab Group toggle
  3. Relaunch

Actual result

If disable the toggle and relaunch Brave it still stays on enabled

Expected result

Just disable that.

Issue reproduces how often

It never disables

Version/Channel Information:

Haven't tried yet

  • Can you reproduce this issue with the current Play Store version? Yes
  • Can you reproduce this issue with the current Play Store Beta version? Haven't tried
  • Can you reproduce this issue with the current Play Store Nightly version? Yes

Device details

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

Brave version

Version 1.25.69

Website problems only

  • Does the issue resolve itself when disabling Brave Shields? No
  • Does the issue resolve itself when disabling Brave Rewards? No
  • Is the issue reproducible on the latest version of Chrome? Haven't tried

Additional information

I hate this tab group feature in phone

@mizzunet mizzunet added the OS/Android Fixes related to Android browser functionality label May 30, 2021
@srirambv
Copy link
Contributor

Reproduced the issue. Toogle doesn't seem to stick gets reenabled after relaunch.

cc: @samartnik

@srirambv
Copy link
Contributor

srirambv commented Jun 8, 2021

Verification passed on OnePlus 6T with Android 10 running 1.25.71 x64 build

  • Verified on a clean install tab-group settings is enabled by default
  • Verified on a clean install open link in tab-group works as expected
  • Verified disabling the tab-group options triggers relaunch prompt
  • Verified disabling the tab-group option and restarting keeps the setting intact and disable gird view in tab tray
  • Verified after tab-group is disabled, context menu doesn't show Open link in a tab-group option
  • Verified re-ordering tabs and disabling tab-group options doesn't reset the tabs positions
  • Verified re-ordering tabs and re-enabling tab-group option regroups all the tabs as before
  • Verified if tab-group is enabled on older build, upgrading to 1.25.71 build retains the settings and retains tab groups
  • Verified if tab-group is disabled on older build, upgrading to 1.25.71 build retains the setting in disabled state and shows tabs in stack layout
  • Verified tab-grouping works on both normal and private tabs when enabled
  • Verified tab stack layout works fine on both normal and private tabs when disabled
  • Verified closing a tab group in tab tray and click on Undo restores the tab group
  • Verified no Group tabs option is shown in tab-tray view when menu is opened
  • Verified able to drag and drop tabs in groups from tab-tray when settings is enabled
  • Verified able to open custom tabs when tab-group setting is enabled or disabled
  • Verified able to open New tab or private tab from home screen
  • Verified no crash when setting is enabled or disabled
  • Verified able to create a new tab inside a tab group via new tab button in tab-group bar and via tab-tray grid view
  • Verified able to swipe switch tabs within tab-group and outside tab-group by swiping the bottom toolbar
  • Verified tab-groups or stack layout works fine with bottom toolbar disabled
image image

Verification passed on Samsung Tab A with Android 10 running 1.25.71 x64 build

  • Verified no tab-group setting is shown for tablets
  • Verified no context menu option for tab-group is shown on long press on a link

Android.Tab.Tab-groups.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA/Yes regression release/blocking release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants