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

Changing orientation doesn't show bottom toolbar #14211

Closed
srirambv opened this issue Feb 17, 2021 · 5 comments
Closed

Changing orientation doesn't show bottom toolbar #14211

srirambv opened this issue Feb 17, 2021 · 5 comments
Assignees
Labels
bug closed/by-author OS/Android Fixes related to Android browser functionality priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include

Comments

@srirambv
Copy link
Contributor

Description

Changing orientation doesn't show bottom toolbar

Steps to reproduce

  1. Launch browser in landscape mode
  2. Visit a page in landscape mode
  3. Rotate to portrait
  4. Bottom toolbar is missing, menu and tab count shows up on top
  5. Rotate to landscape and back to portrait, bottom toolbar shows up

Actual result

Bottom.toolbar.issue.mp4

Expected result

Should not hide bottom toolbar when orientation is changed

Issue reproduces how often

Easy

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? Yes
  • 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): Phone
  • Android version: 10

Brave version

1.20.103

Website problems only

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

Additional information

cc: @samartnik

@srirambv srirambv added bug QA/Yes release-notes/include OS/Android Fixes related to Android browser functionality labels Feb 17, 2021
@samartnik
Copy link
Contributor

Seems to be not reproducible on my devices. @srirambv did you do anything else prior to these steps?

@srirambv
Copy link
Contributor Author

@samartnik no specific steps you can try the following which repro's 90% of the time

  • Clean install any build (do not launch)
  • Change device orientation to landscape
  • Launch Brave in landscape orientation
  • Visit a page
  • Change orientation to portrait
  • Bottom toolbar is missing
  • Change to landscape and back to portrait shows bottom toolbar

Here's a recording on PlayStore version (Do not the onboarding modals force rotation so closing them and showing. It even happens with P3A onboarding is shown)

14211.mp4

@Revenrof
Copy link

@samartnik regarding bottom toolbar orientation, I got "bottom toolbar" thingy in landscape mode IF, I open "tabs switcher".

In NTP landscape mode the bottom toolbar shown in split second and then puff. Gone.

But, IF the tab is used to open any url (including "chrome://what-do-I-call-this-?") the bottom toolbar is shown.

  • tab opening "https://brave.com" :
    Screenshot_20210219-063631

  • tab opening "chrome://version" :
    Screenshot_20210219-065046

  • tab opening "about:blank" :
    Screenshot_20210219-070041

However, none of them ( the buttons) is working. Is it supposed to be like this? Do I need to open new issue?

@samartnik
Copy link
Contributor

samartnik commented Feb 19, 2021

@Revenrof it is not expected behavior as bottom toolbar shouldn't be visible in landscape mode. Yes, please open new issue as this one is different.
upd: actually I created issue myself, thank you for reporting!

@srirambv
Copy link
Contributor Author

srirambv commented Jun 3, 2021

Unable to repro the issue on nightly/beta/release. Closing the issue for now as invalid. Will reopen if I can get it to reproduce

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug closed/by-author OS/Android Fixes related to Android browser functionality priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include
Projects
None yet
Development

No branches or pull requests

4 participants