Skip to content

Text/content cuts off the widget in the new tab page #13537

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.

Sign up for GitHub

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

Jump to bottom
Closed
GeetaSarvadnya opened this issue Jan 13, 2021 · 2 comments · Fixed by brave/brave-core#7590
Closed

Text/content cuts off the widget in the new tab page #13537

GeetaSarvadnya opened this issue Jan 13, 2021 · 2 comments · Fixed by brave/brave-core#7590
Labels
OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Test-Plan-Specified QA/Yes regression release/blocking release-notes/exclude
Milestone
1.20.x - Release

Comments

@GeetaSarvadnya
Copy link

Description

Text/content cuts off the widget in the new tab page

Steps to reproduce

  1. Clean or upgraded profile 1.20.74
  2. Click on Continue on P3A onboarding screen
  3. Open a NTP
  4. Text/content cuts off the widget in the new tab page

Actual result

image
image
image

Expected result

Screenshot_2021-01-13-13-39-50
Screenshot_2021-01-13-13-49-37
Screenshot_2021-01-13-13-49-40

Issue reproduces how often

Easy

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? No
  • Can you reproduce this issue with the current Play Store Beta version? No
  • 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): Samsung Galaxy j3 version 5.1.1
  • Android version: 5.1.1

Brave version

1.20.74

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: @brave/legacy_qa @deeppandya @samartnik @SergeyZhukovsky
@GeetaSarvadnya GeetaSarvadnya added QA/Yes QA/Test-Plan-Specified regression OS/Android Fixes related to Android browser functionality labels Jan 13, 2021
@GeetaSarvadnya GeetaSarvadnya added this to the 1.20.x - Nightly milestone Jan 13, 2021
This was referenced Jan 13, 2021
Merged
Closed
@deeppandya deeppandya mentioned this issue Jan 13, 2021
Merged
23 tasks
@kjozwiak
Copy link
Member

Closing as I believe this was fixed by brave/brave-core#7590. @srirambv @GeetaSarvadnya please re-open if that's not the case 👍

@kjozwiak kjozwiak mentioned this issue Jan 22, 2021
Merged
4 tasks
@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Feb 4, 2021 •
edited
Loading

Verification passed on Samsung Galaxy J3 version 5.1.1 running Bravearm 1.20.98

Screenshot_2021-02-04-17-53-12
Screenshot_2021-02-04-17-53-22
Screenshot_2021-02-04-17-53-25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Test-Plan-Specified QA/Yes regression release/blocking release-notes/exclude
Projects
None yet
Milestone
1.20.x - Release
Development

Successfully merging a pull request may close this issue.

Update NTP code to avoid ID related issue brave/brave-core
3 participants
@kjozwiak @deeppandya @GeetaSarvadnya