Update favicons for light and dark modes - #1829
Conversation
|
Hey @davinotdavid, when I try out the branch locally, the Appointment tab (active or inactive) only always shows in light mode. If I set my Firefox browser settings to Dark mode, or I set the Appointment theme settings to Dark mode, the browser tab still remains in light mode. Is this working for you? |
Oh wait, I see when I change my overall MacBook Apple theme to Dark mode then the Appointment tab does change into dark mode (active or inactive tab). Is that the expected behaviour? Are there plans to implement this at the Browser level or Appointment theme level too, or just System level for now? |
Ah, part of #1821 will be to actually remove the user setting of theme within Appointment altogether (will make a note for me to remember to remove the tests related to it), in favor of device settings first + toggle in the navbar itself. The browser level / user agent should work though 🤔 . I wonder if this is a Firefox bug of sorts? We are using the standard |
devmount
left a comment
There was a problem hiding this comment.
I tested this in Firefox and Helium on Tuxedo OS with both, dark and light mode settings in the browser. Perfectly working fine on my end.
|
Thanks @rwood-moz and @devmount for the great testing / reviews! 🙏 |



Light mode (active tab):

Light mode (inactive tab):

Dark mode (active tab):

Dark mode (inactive tab):

Changed per Design team request, based off of these Figma files. Title has already been changed!
Closes #1820