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

two different relaunch buttons being displayed at times under brave://settings/rewards #19675

Closed
kjozwiak opened this issue Nov 23, 2021 · 3 comments · Fixed by brave/brave-core#15023

Comments

@kjozwiak
Copy link
Member

Description

Sometimes two different type of Relaunch buttons will be used/displayed under brave://settings/rewards when enabling/displaying multiple settings under Tip buttons.

Steps to Reproduce

  1. go into brave://settings/rewards
  2. enable/disable GH/Twitter/Reddit a few times (try a few combinations of the enabled/disabled at the same time)

You'll eventually run into an issue where you'll have two sets of different buttons

Actual result:

relaunchIssue

Expected result:

image

Reproduces how often:

I would say there's ~35% or so change of reproducing. It doesn't always happen and I've only seen it a few times

Brave version (brave://version info)

Brave | 1.34.28 Chromium: 96.0.4664.45 (Official Build) nightly (64-bit)
-- | --
Revision | 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS | Windows 11 Version 21H2 (Build 22000.318)

Version/Channel Information:

Other Additional Information:

  • 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

Miscellaneous Information:

CCing @Miyayes @LaurenWags @emerick

@emerick
Copy link
Contributor

emerick commented Nov 23, 2021

@kjozwiak Yes, I noticed this behavior as well. I'm wondering if it's an upstream bug with the button stying (though I can't find a ticket for it if it is), because I notice the same behavior with other settings like "Allow Google login for extensions" on brave://settings/extensions.

@emerick
Copy link
Contributor

emerick commented Nov 23, 2021

It seems like our Polymer overrides for cr-button don't work the second time you trigger them. Simple repro:

  1. Visit brave://settings/extensions
  2. Enable "Allow Google login for extensions" toggle
  3. Relaunch button appears with appropriate styling
  4. Disable "Allow Google login for extensions" toggle
  5. Relaunch button disappears
  6. Enable "Allow Google login for extensions" toggle
  7. Relaunch button appears with inappropriate (default, I suppose) styling

@petemill Not sure if you might be familiar with what's happening here?

@zenparsing zenparsing added the priority/P4 Planned work. We expect to get to it "soon". label Nov 24, 2021
@emerick emerick added this to the 1.45.x - Nightly milestone Sep 8, 2022
@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 27, 2022
@stephendonner
Copy link

stephendonner commented Sep 27, 2022

Verification PASSED using

Brave 1.45.75 Chromium: 106.0.5249.65 (Official Build) beta (x86_64)
Revision 3269dc3633cdd2ab94546fdbe54962e45b17a6e0-refs/branch-heads/5249@{#580}
OS macOS Version 11.7 (Build 20G817)

Easily reproduced both issues using 1.44.101

brave://settings/rewards brave://settings/extensions
Screen Shot 2022-09-27 at 6 45 45 PM Screen Shot 2022-09-27 at 6 48 15 PM

Steps:

  1. installed 1.45.75
  2. launched Brave
  3. opened brave://settings/rewards
  4. enabled Rewards by clicking on Brave Rewards are disabled
  5. click on Start using Brave Rewards
  6. dismiss the tour
  7. under Tip buttons, toggle GitHub to Disabled
  8. confirmed a Relaunch button appears
  9. toggled Reddit and Twitter back and forth a bunch of times

Screen Shot 2022-09-27 at 7 04 01 PM

Spun off the 2nd instance of this issue mentioned as #25661.

@stephendonner stephendonner added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment