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

[Android] Bottom toolbar setting reverted (possibly on update) #9843

Closed
mbacchi opened this issue May 19, 2020 · 7 comments · Fixed by brave/brave-core#5880
Closed

[Android] Bottom toolbar setting reverted (possibly on update) #9843

mbacchi opened this issue May 19, 2020 · 7 comments · Fixed by brave/brave-core#5880

Comments

@mbacchi
Copy link
Contributor

mbacchi commented May 19, 2020

Description

I have "Enable bottom toolbar" checked, and when updating the last time it was reverted. I also noticed it has reverted today whether there was an update or not I'm not sure.

Steps to reproduce

  1. Turn "Enable bottom toolbar" on
  2. Wait for update (or maybe just wait a number of days not sure my version was updated during this latest reversion)
  3. Setting is reverted, toolbar is on top again

Actual result

Expected result

Toolbar should remain on bottom

Issue reproduces how often

Issue happens on

  • Current Play Store version?
  • Beta build?

Device details

  • Install type (ARM, x86): ?
  • Device (Phone, Tablet, Phablet): Pixel 2
  • Android version: 10 (QQ2A.200501.001.B3)

Brave version

1.8.112 (81.0.4044.138)

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

@mbacchi mbacchi added the OS/Android Fixes related to Android browser functionality label May 19, 2020
@simonhong
Copy link
Member

I also had the same issue with stable release.

@anthonypkeane
Copy link

Screen Shot 2020-06-12 at 08 49 30

@LaurenWags
Copy link
Member

I haven't reproduced this in testing. I do have Beta and Stable installed on my personal device from the GPS. When Beta updates this never happens, but when Stable updates this occurs for me also.

@srirambv
Copy link
Contributor

I haven't seen the issue on Pixel 3XL on release. Bottom bar is always retained.

@SergeyZhukovsky
Copy link
Member

I'm going to remove upgrade logic from tabs to brave-core based. For QA team to check that you will need to get a latest tabs apk install it, change settings that we migrate, background video playback for example or view sites on desktop mode always and make sure that on upgrade to brave-core based they are not retained.

@nickidw
Copy link

nickidw commented Jun 21, 2020

I'm glad some other people are having the same issue, I thought I was going mad!

@GeetaSarvadnya
Copy link

Verification passed on Samsung Galaxy J3 version 5.1.1 running BraveModernarm.apk.1.10.95

  • Verified upgrading from 1.10.94 to 1.10.95 doesn't reset bottom toolbar state
  • Verified upgrading from 1.9.80 to 1.10.95 doesn't reset bottom toolbar state

Verification passed on Gionee version 5.1 running BraveModernarm.apk.1.10.95

  • Verified upgrading from 1.10.94 to 1.10.95 doesn't reset bottom toolbar state
  • Verified upgrading from 1.9.80 to 1.10.95 doesn't reset bottom toolbar state

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants