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

Shields panel doesn't always update immediately upon light/dark-mode toggle #22389

Closed
Tracked by #22546
stephendonner opened this issue Apr 19, 2022 · 6 comments
Closed
Tracked by #22546
Labels
bug feature/global-settings Settings at browser level independent of shields settings feature/shields/panel Front-end design and functionality of the Shields panel. feature/shields The overall Shields feature in Brave. OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-All-Platforms QA/Test-Plan-Specified QA/Yes release-notes/exclude

Comments

@stephendonner
Copy link

stephendonner commented Apr 19, 2022

Description

Shields panel doesn't always update immediately upon light/dark-mode toggle

Steps to Reproduce

  1. install 1.39.68
  2. launch Brave
  3. load twitch.tv
  4. click on the Shields icon in the URL bar
  5. in a new tab, open brave://settings/appearance
  6. if Light, choose Dark, and vice-versa
  7. in the twitch.tv tab, click again on the Shields icon in the URL bar

Actual result:

Screen Shot 2022-04-18 at 8 13 20 PM

Expected result:

The Shields panel should observe pref changes to Light / Dark and re-validate its cache

Screen Shot 2022-04-18 at 8 14 55 PM

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.39.68 Chromium: 101.0.4951.34 (Official Build) nightly (x86_64)
Revision 8f93af5d759634c5256b65997dedd138ed60992f-refs/branch-heads/4951@{#710}
OS macOS Version 12.4 (Build 21F5048e)

cc @nullhook @rebron @MadhaviSeelam @GeetaSarvadnya

@stephendonner stephendonner added bug feature/shields The overall Shields feature in Brave. feature/global-settings Settings at browser level independent of shields settings QA/Yes QA/Test-Plan-Specified feature/shields/panel Front-end design and functionality of the Shields panel. OS/Desktop labels Apr 19, 2022
@stephendonner
Copy link
Author

@simonhong wonder if you have a moment to take a look? 🙏

@simonhong
Copy link
Member

This happens because shields bubble is destroyed after some delay.
After some times passed(don't know the exact timing..), shields bubble will have proper theme color.
@nullhook Do you think shiels panel can monitor theme change and use it?

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Apr 22, 2022
@rebron rebron mentioned this issue Apr 25, 2022
7 tasks
@rebron rebron added the needs-investigation A bug not 100% confirmed/fixed label Apr 26, 2022
@rebron
Copy link
Collaborator

rebron commented Jun 1, 2022

Closed with brave/brave-core#13209

@rebron rebron closed this as completed Jun 1, 2022
@rebron rebron added this to the 1.41.x - Nightly milestone Jun 1, 2022
@rebron rebron added release-notes/exclude and removed needs-investigation A bug not 100% confirmed/fixed labels Jun 1, 2022
@stephendonner
Copy link
Author

Verified PASSED using

Brave 1.41.40 Chromium: 102.0.5005.78 (Official Build) nightly (x86_64)
Revision df6dbb5a9fd82af3f567198af2eb5fb4876ef99c-refs/branch-heads/5005_59@{#3}
OS macOS Version 12.5 (Build 21G5037d)
example example example
Screen Shot 2022-06-06 at 5 30 46 PM Screen Shot 2022-06-06 at 5 31 03 PM Screen Shot 2022-06-06 at 5 31 09 PM

@MadhaviSeelam
Copy link

Verification Passed using

Brave | 1.41.50 Chromium: 102.0.5005.99 (Official Build) nightly (64-bit)
-- | --
Revision | 870f7bcc58dfa811cc68c2186439721385e086d0-refs/branch-heads/5005@{#1125}
OS | Windows 11 Version 21H2 (Build 22000.675)
Image1 Image2 Image3
image1 image2 image1

@stephendonner
Copy link
Author

Verified PASSED using

Brave 1.41.77 Chromium: 103.0.5060.53 (Official Build) beta (64-bit)
Revision a1711811edd74ff1cf2150f36ffa3b0dae40b17f-refs/branch-heads/5060@{#853}
OS Linux
example example example
Screen Shot 2022-06-23 at 3 03 06 PM Screen Shot 2022-06-23 at 3 03 14 PM Screen Shot 2022-06-23 at 3 03 19 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/global-settings Settings at browser level independent of shields settings feature/shields/panel Front-end design and functionality of the Shields panel. feature/shields The overall Shields feature in Brave. OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-All-Platforms QA/Test-Plan-Specified QA/Yes release-notes/exclude
Projects
None yet
Development

No branches or pull requests

4 participants