Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

UI sometimes freezes when Rewards are enabled #3941

Closed
iccub opened this issue Jul 19, 2021 · 6 comments
Closed

UI sometimes freezes when Rewards are enabled #3941

iccub opened this issue Jul 19, 2021 · 6 comments

Comments

@iccub
Copy link
Contributor

iccub commented Jul 19, 2021

Description:

We've got few reports from users that sometimes when you visit a page with Rewards ON, the UI freezes for few seconds.
This happens on high end devices too.
I personally was unable to reproduce it.

Few websites that this happens on is Twitter, ESPN, CNN

Steps to Reproduce

  1. Rewards ON
  2. Start using the app, visiting pages.

Actual result:

Expected result:

Reproduces how often: [Easily reproduced, Intermittent Issue]

Brave Version:

Device details:

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@iccub iccub added this to the 1.30 milestone Jul 19, 2021
@bsclifton
Copy link
Member

Internal notes (employee only) can be found here https://bravesoftware.slack.com/archives/C06UXF3KJ/p1625962370289400

@iccub
Copy link
Contributor Author

iccub commented Sep 29, 2021

linked brave-core ticket here brave/brave-browser#18118

@kjozwiak
Copy link
Member

kjozwiak commented Sep 29, 2021

Cases were also added via brave/brave-core#10088 (comment). @srirambv we'll need to double check with both @bsclifton & @jsecretan to see if they can still reproduce the issue once brave/brave-core#10088 makes it into 1.32.x.

@kjozwiak
Copy link
Member

Closing as brave/brave-core#10280 was merged. We'll need to wait till @kylehickinson or @iccub can you let us know via this issue when we get a TF build that includes the above? Thanks.

@iccub
Copy link
Contributor Author

iccub commented Sep 29, 2021

I will let you know @kjozwiak and put brave-core library info in TF build description.

@srirambv
Copy link
Contributor

@iccub confirmed that this is fixed and works fine now for both @jsecretan and @bsclifton . Since its pretty hard to repro and we have confirmation internally that its working fine, marking the issue as QA/No

@srirambv srirambv added QA/No and removed QA/Yes labels Oct 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants