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

NTP Stats Toggle is Broken #24985

Closed
fallaciousreasoning opened this issue Aug 25, 2022 · 6 comments · Fixed by brave/brave-core#14839
Closed

NTP Stats Toggle is Broken #24985

fallaciousreasoning opened this issue Aug 25, 2022 · 6 comments · Fixed by brave/brave-core#14839

Comments

@fallaciousreasoning
Copy link

fallaciousreasoning commented Aug 25, 2022

Description

The stats toggle in the customise NTP doesn't update. You can still use it to reshow the stats, and hide them via the 3 dot menu but the toggle isn't working properly.

Screenshot 2022-08-26 at 12 43 48 PM

@stephendonner
Copy link

stephendonner commented Aug 26, 2022

Confirmed using

Brave 1.44.60 Chromium: 105.0.5195.37 (Official Build) nightly (x86_64)
Revision 5f67e9f258cc28ab970e57572290edf1863ee3c5-refs/branch-heads/5195@{#621}
OS macOS Version 13.0 (Build 22A5321d)

Screenshot 2022-08-25 at 5 22 55 PM

Steps:

  1. install 1.44.60
  2. launch Brave
  3. open a new tab/window
  4. click on Customize on the lower right
  5. click on Brave Stats
  6. look at the Show Brave Stats button

Actual:

It's disabled

Expected:

It should be on/off toggleable (sp!)

@stephendonner
Copy link

Also broken in

Brave 1.45.1 Chromium: 105.0.5195.37 (Official Build) nightly (x86_64)
Revision 5f67e9f258cc28ab970e57572290edf1863ee3c5-refs/branch-heads/5195@{#621}
OS macOS Version 13.0 (Build 22A5321d)

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Aug 26, 2022
@brave-builds brave-builds added this to the 1.45.x - Nightly milestone Aug 28, 2022
@stephendonner
Copy link

stephendonner commented Sep 2, 2022

Verified PASSED using

Brave 1.45.21 Chromium: 105.0.5195.68 (Official Build) nightly (x86_64)
Revision ad13e82529051bac6a0e65f455e6d7a1e5fd7938-refs/branch-heads/5195@{#903}
OS macOS Version 13.0 (Build 22A5331f)

Steps:

  1. launch Brave
  2. open a new tab/window
  3. click on Customize on the lower right
  4. click on Brave Stats
  5. look at the Show Brave Stats button
  6. toggle it Off
  7. toggle it On

Confirmed Off / On state correlated with stats dashboard

toggled Off toggled On
Screenshot 2022-09-01 at 10 46 21 PM Screenshot 2022-09-01 at 10 46 26 PM

@kjozwiak
Copy link
Member

kjozwiak commented Oct 5, 2022

Removed the QA Pass-macOS as this will need to be rechecked on 1.44.x once we get RC builds. I also added release-notes/include as this ended up making it into 1.44.x. Hence, adding it into the 1.44.x maintenance release.

@stephendonner also double checked that the above was fixed with the latest 1.45.x via #25663 (comment) which we used to uplift this into 1.44.x.

@kjozwiak
Copy link
Member

kjozwiak commented Oct 5, 2022

The above requires 1.44.108 or higher for 1.44.x verification. Just a note that DevOps kicked off a 1.44.107 for noplatforms so there was never any platforms builds for that version.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 6, 2022

Verification PASSED on

Brave | 1.44.108 Chromium: 106.0.5249.103 (Official Build) (64-bit)
-- | --
Revision | 182570408a1f25ab2731ef5f283b918df9b9f956-refs/branch-heads/5249_91@{#6}
OS | Windows 10 Version 21H2 (Build 19044.2006)
  • Reproduced the issue in 1.44.101 , 1.44.103 and 1.44.105
  • Upgraded the profile to 1.44.108 and ensured Show Brave Stats button can be toggled on/off
Example Example Example Example Example Example
image image image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants