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

Add followSystemTheme config option #1082

Merged
merged 7 commits into from
Jan 30, 2024

Conversation

benterova
Copy link
Contributor

  • New command line option --followSystemTheme. Defaults to false.
  • When following system theme, set Teams theme to the current light/dark setting after a small delay on startup. (2.5 seconds, if you attempt to do this too soon Teams freaks out and only updates the outer window with the new theme)
  • Update Teams light/dark theme to match system theme when the system theme is changed.

@jijojosephk
Copy link
Collaborator

@benterova thanks for your efforts ❤️ could you update the package.json with version number? I think, it would be 1.4.6

@benterova
Copy link
Contributor Author

@benterova thanks for your efforts ❤️ could you update the package.json with version number? I think, it would be 1.4.6

Done 😄 Let me know if there's anything else that needs to be done!

@jijojosephk
Copy link
Collaborator

@benterova sorry, there was a merge in between. Can you sync your fork and update once again please? The version was changed to 1.4.6 already.

@benterova
Copy link
Contributor Author

@benterova sorry, there was a merge in between. Can you sync your fork and update once again please? The version was changed to 1.4.6 already.

Updated again

@jijojosephk
Copy link
Collaborator

@benterova it shows merge conflict. I guess you have to sync your fork and push again.

@jijojosephk jijojosephk merged commit cc5e376 into IsmaelMartinez:develop Jan 30, 2024
4 checks passed
@jijojosephk
Copy link
Collaborator

Thanks @benterova. I fixed the conflict and merged. Thanks again! I'm sure the community will be very happy about this new addition.

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.

2 participants