feat: polls in chat#62
Conversation
|
merged develop since the test was fixed (sorta) |
dankgarlic1
force-pushed
the
feat/polling-in-meeting-chat
branch
from
296de0b to
05c64ee
Compare
|
jam: https://jam.dev/c/e8b82540-19be-4deb-96a6-2cb94a40dc6c the color scheme sucks and I am not sure what to have, can you suggest like on the overall ui what has to be changed? |
BreadGenie
left a comment
BreadGenie
left a comment
There was a problem hiding this comment.
Agreed on the UI. Especially the card inside card UI feels a bit vibed. I'd rather we use black and white theme as usual in espresso design. Give it a shot without other colours. You can take a look at other products for inspiration.
Also, we need to fix the inconsistency in the terminology. Let's stick with create poll instead of launch poll.
Few additional things I'd like to see
- View who voted. Currently it looks anonymous.
- Poll deadline
These can be added in this PR or a separate one.
|
I feel these things should be done in a separate pr
will create issues for that rest yes I'll give it a shot for ui, thanks |
|
btw what design inspirations you took while making meet? |
I went with whatever I thought made sense. I'll do a redesign very soon btw. Because a lot of things are inconsistent. |
dankgarlic1
force-pushed
the
feat/polling-in-meeting-chat
branch
from
05c64ee to
68ef7cc
Compare
|
@BreadGenie hey I updated the color scheme: https://jam.dev/c/c42e3c51-6258-4f16-99b5-f8bff43f2a39 I like it, its simple and just nice matches the vibe of all the frappe products, lmk what you think |
|
although I do feel the way that drop down is positioned is idk not the best I feel, lmk what you think, happy to rework on it if needed |
2 participants
closes #58