Skip to content

Update favicons for light and dark modes - #1829

Merged
davinotdavid merged 1 commit into
mainfrom
davinotdavid/update-favicons
Sep 15, 2026
Merged

davinotdavid merged 1 commit into
mainfrom
davinotdavid/update-favicons

Conversation

@davinotdavid

@davinotdavid davinotdavid commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Light mode (active tab):
image

Light mode (inactive tab):
image

Dark mode (active tab):
image

Dark mode (inactive tab):
image

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

Closes #1820

@rwood-moz

Copy link
Copy Markdown
Contributor

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?

@rwood-moz

Copy link
Copy Markdown
Contributor

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?

@davinotdavid

Copy link
Copy Markdown
Contributor Author

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 prefers-color-scheme API to switch the colours on it.

@devmount devmount left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@rwood-moz

Copy link
Copy Markdown
Contributor

Ok I see the issue - but it's a Firefox/Mac issue, not an Appointment issue. FYI I'm on MacBook M3 Pro Tahoe 26, with latest Firefox 156.0.

The issue is that when I change the Firefox browser setting to dark mode, the browser tabs themselves do NOT change to dark mode.

Screenshot with Firefox browser light mode:

Screenshot 2026-09-15 at 11 43 38 AM

Screenshot with Firefox browser settings set to dark, notice the tabs remain light:

Screenshot 2026-09-15 at 11 43 47 AM

For my Firefox, the only way to get the browser tabs themselves to change to dark mode is to change the Apple system theme setting:

Screenshot 2026-09-15 at 11 44 10 AM

So this is not an Appointment issue since it happens with all the tabs.

@davinotdavid

Copy link
Copy Markdown
Contributor Author

Thanks @rwood-moz and @devmount for the great testing / reviews! 🙏

@davinotdavid
davinotdavid merged commit 31c2fa3 into main Sep 15, 2026
10 checks passed
@davinotdavid
davinotdavid deleted the davinotdavid/update-favicons branch September 15, 2026 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update favicon + meta title

3 participants