- Notifications
You must be signed in to change notification settings - Fork928
feat: Add emoji picker to group settings#4685
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
"--rgb-background": theme.palette.background.paper, | ||
"--rgb-input": colors.gray[17], | ||
"--rgb-color": colors.gray[4], | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Do we want to add thissite/src/theme/overrides.ts
? I feel like I might not look for it here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I just copied this from the template settings one, but I assume we'll abstract this (maybe once it's used once more?).
Kira-PilotOct 21, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I could be wrong, but I thought the@global
key meant any classes defined within would be defined globally. So, if this is already declared in template settings, I'm surprised we need it here, too.
I'm suggesting we move it tooverrides
because we have a bunch of other globals there, and it's nice to have them colocated. But not a blocker.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I believe the@global
will go away when the component is unmounted, so it won't actually apply globally to all pages, just to the global scope on this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Ohhh, interesting; thanks for explaining. Alrighty, let's keep it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
nice!
Fixes#4413.