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

Browser Menus do not Respond to Light/Dark OS Theme Changes #1506

Closed
jhreis opened this issue Sep 7, 2019 · 2 comments · Fixed by #3606
Closed

Browser Menus do not Respond to Light/Dark OS Theme Changes #1506

jhreis opened this issue Sep 7, 2019 · 2 comments · Fixed by #3606

Comments

@jhreis
Copy link
Contributor

jhreis commented Sep 7, 2019

Description:

Most menus inside Brave to do not respond to OS dark / light theme changes.

From a technical perspective, this is an opportunity to clean up menu usage through out application and make parent -> child UI relationships more definitive.

When BVC gets a trait collection update (BVC responds correctly), it should be able to propagate an applyTheme update down to all presented VC's and children. This includes having a handle on settings, and all nested menus.

This also breaks the tight coupling we often have throughout the application where children often dismiss themselves (meaning they know how they are being presented).

Steps to Reproduce

  1. Make sure theme selection is "automatic"
  2. Open settings (or main menu / shields panel)
  3. Change the OS system theme
  4. Observe settings screen is not dynamically updated (also happens with other menus)

Actual result:

Image of wrong colored main menu when theme changed
Image of wrong colored shield panel when theme changed

Expected result:

All menus should be styled correctly.

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

Related #1503

@srirambv
Copy link
Contributor

srirambv commented Sep 9, 2019

This is also a problem when app is in settings page and you try to change theme from action menu. Recording in #1503 (comment)

@srirambv
Copy link
Contributor

Verification passed on iPhone XR with iOS 13.5 running 1.26(21.5.25.18)

  • Verified steps from issue description
  • Verified theme updates correctly for menu/shields/rewards while open, when theme is changed via control centre

Verification passed on iPhone 7+ with iOS 14.5.1 running 1.26(21.5.25.18)

  • Verified steps from issue description
  • Verified theme updates correctly for menu/shields/rewards while open, when theme is changed via control centre

Verification passed on iPad Pro with iOS 14.5.1 running 1.26(21.5.25.18)

  • Verified steps from issue description
  • Verified theme updates correctly for menu/shields/rewards while open, when theme is changed via control centre

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.