question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

NTP becomes scrollable and `Load new content` appears when changing device orientation when Brave News disabled

See original GitHub issue

Description

While running through https://github.com/brave/brave-browser/issues/22135 on 1.38.x, I noticed that the NTP page becomes scrollable when changing the orientation from vertical -> horizontal -> vertical and the Load new content button appears via the NTP even though Brave News has never been disabled.

Steps to reproduce

Test Case #1 (in this case, you’ll get the Load new content button appearing)

  1. install a version of Brave that has the opt-in card visible by default via NTP (used 1.38.107 Chromium: 101.0.4951.41)
  2. disable the Brave News opt-in card via the NTP by pressing the X button
  3. once the opt-in card has been disabled, changed the orientation from vertical -> horizontal -> vertical
  4. notice that the NTP is scrollable and when you start scrolling, you’ll see the Load new content button appear

Test Case #2 (in this case, the NTP becomes scrollable but the Load new content button does’t appear)

  1. install a version of Brave that has the opt-in card visible by default via NTP (used 1.38.107 Chromium: 101.0.4951.41)
  2. opt-in into Brave News via the opt-in card under the NTP
  3. once enabled, disable Brave News via Settings
  4. change the orientation from vertical -> horizontal -> vertical
  5. notice that the NTP is scrollable even though Brave News has been disabled

Note: opening a new tab resolves the above. The Load new content button doesn’t appear across different NTP. However, it’s 100% reproducible on every NTP page and only happens when changing orientations.

Actual result

https://user-images.githubusercontent.com/2602313/165386726-fca32b32-58de-4922-9219-ef833eda36bc.mp4

Expected result

Changing the orientation of the device shouldn’t make the NTP tab scrollable nor display the floating Load new content button.

Issue reproduces how often

100% reproducible using the STR/Cases outlined above.

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? No (1.38.x hasn’t been released yet)
  • 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): Samsung S10+
  • Android version: Android 12

Brave version

  • 1.38.107 Chromium: 101.0.4951.41

Website problems only

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

Additional information

Issue Analytics

  • State:closed
  • Created a year ago
  • Comments:5 (4 by maintainers)

github_iconTop GitHub Comments

1reaction
kjozwiakcommented, Apr 26, 2022

This needs to be completed for the 1.38.x maintenance release so we can re-enable https://github.com/brave/brave-browser/issues/22135.

0reactions
kjozwiakcommented, May 16, 2022

Verification PASSED on Samsung S10+ running Android 12 using the following build(s):

Brave | 1.38.118 Chromium: 101.0.4951.67 (Official Build) (64-bit)
--- | ---
Revision | 8888ee7a24e2c36661ddb9536c35b7d4852a3a98-refs/branch-heads/4951@{#1230}
OS | Android 12; Build/SP1A.210812.016

Went through the STR/Cases outlined via https://github.com/brave/brave-browser/issues/22569 and ensured that:

  • the NTP isn’t scrollable like before while viewing NTP in both portrait & landscape modes
  • ensured that changing the orientation doesn’t display the Load new content button while Brave News is disabled

https://user-images.githubusercontent.com/2602313/168521185-cf15a617-0011-4f3a-b394-ff3c17ce4e36.mp4

https://user-images.githubusercontent.com/2602313/168521140-b99f3837-079d-4b49-9931-15dacecbfea0.mp4

Verification PASSED on Samsung Galaxy Tablet A running Android 11 using the following build(s):

Brave | 1.38.118 Chromium: 101.0.4951.67 (Official Build) (64-bit)
--- | ---
Revision | 8888ee7a24e2c36661ddb9536c35b7d4852a3a98-refs/branch-heads/4951@{#1230}
OS | Android 11; Build/RP1A.200720.012

Went through the STR/Cases outlined via https://github.com/brave/brave-browser/issues/22569 and ensured that:

  • the NTP isn’t scrollable like before while viewing NTP in both portrait & landscape modes
  • ensured that changing the orientation doesn’t display the Load new content button while Brave News is disabled

https://user-images.githubusercontent.com/2602313/168521499-a7a0a822-61c3-4d85-a06a-518a63f980af.mp4

https://user-images.githubusercontent.com/2602313/168521735-c87d3456-61d8-4bcb-afa2-a9abcfea47bb.mp4

Read more comments on GitHub >

github_iconTop Results From Across the Web

crash changing orientation/scrolling through NTP after ...
Brave shouldn't be crashing and the NTP shouldn't be scrollable when news has been disabled and a user is using in landscape mode....
Read more >
Brave turns itself off when I scroll down on Brave News
Just a moment ago, while scrolling down on Brave News, an option called 'upload new contents' appeared at the top (this is the...
Read more >
Can I disable scrolling up for Brave News on the new tab page?
Update: The forum thread linked below has a fix: Open brave://flags/, search for news, disable "Brave News prompts on New Tab Page".
Read more >
Solved: Canvas app only appears without scrollbars if Chro...
DesignHeight, any time the screen height was less than the App.DesignHeight the vertical scrollbar appeared which caused the horizontal scrollbar to appear.
Read more >
Untitled
Daniel rikiriki, 2005 winnebago voyage 33v brochure, Dover nh latest news. Svt raptor wheels and tires, Todos os jogos do naruto de xbox...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found