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 dark mode #136

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Add dark mode #136

merged 2 commits into from
Mar 14, 2024

Conversation

billykern
Copy link
Contributor

Addresses: #129

Please let me know if you would prefer to adjust the color scheme or toggle position...

New dark theme:
Screenshot 2024-03-14 at 4 13 06 PM

Regression light theme:
Screenshot 2024-03-14 at 4 14 44 PM

New Option:
Screenshot 2024-03-14 at 4 15 03 PM

const typingIndicatorStyle = {
backgroundColor: theme.palette.background.paper,
color: theme.palette.text.primary,
};
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if there is a better way to apply the MUI themes, this was the best I could come up with, but let me know if you have another idea

@@ -67,6 +67,7 @@
"eslint-plugin-react": "^7.33.2",
"eslint-plugin-react-hooks": "^4.6.0",
"html-webpack-plugin": "^5.5.3",
"jest-environment-jsdom": "^29.7.0",
Copy link
Contributor Author

@billykern billykern Mar 14, 2024

Choose a reason for hiding this comment

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

apparently this became necessary with jest v28, I had to do this to get around TypeError: Cannot read properties of undefined (reading 'html')

https://stackoverflow.com/a/72180987

@andrewnguonly
Copy link
Owner

Looks great! Thanks @billykern!

@andrewnguonly andrewnguonly merged commit c46176b into andrewnguonly:1.0.11 Mar 14, 2024
1 check passed
@billykern billykern deleted the 1.0.11 branch March 21, 2024 03:58
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.

None yet

2 participants