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] Add missing landscape layout resources for site banner activity. #8587

Closed
gdregalo opened this issue Mar 7, 2020 · 2 comments · Fixed by brave/brave-core#4880
Closed
Assignees
Labels

Comments

@gdregalo
Copy link

gdregalo commented Mar 7, 2020

Description

Please add missing landscape layout resources for site banner activity.

Steps to reproduce

Actual result

Expected result

Issue reproduces how often

Issue happens on

  • Current Play Store version?
  • Beta build?

Device details

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

Brave version

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

@gdregalo
Copy link
Author

gdregalo commented Mar 7, 2020

Screenshot_20200304-181255

@NejcZdovc NejcZdovc added feature/rewards OS/Android Fixes related to Android browser functionality labels Mar 11, 2020
@srirambv
Copy link
Contributor

Verification passed on Samsung Tab A with Android 9.0 running 1.5.111 x64 stable build

  • Verified on landscape more the site banner is shown correctly
    image

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

  • Verified on landscape more the site banner is shown correctly
    image

tmancey pushed a commit that referenced this issue Apr 9, 2020
…pe_1.7.x

Add missing landscape layout resources for site banner activity. #8587 (uplift to 1.7.x)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants