-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat: minimal bar-[INS-4420] #8293
Conversation
aaf06b2
to
297d8cf
Compare
432ec04
to
2eac655
Compare
2eac655
to
9af5816
Compare
9af5816
to
16f9ad0
Compare
9c9ab49
to
6ccddfe
Compare
)} | ||
</div> | ||
{!isMinimal && ( | ||
<div className="flex gap-[--padding-sm] items-center justify-end p-2"> |
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 think we should extract the invite button and user button to standalone components 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.
agree
@cwangsmv Good catch! I updated style, when there is not enough space, the middle column will be compressed. |
ac9375b
to
5c0f9cb
Compare
2480ffd
to
b445fc1
Compare
b445fc1
to
b65a6b3
Compare
57cab7a
to
87bc610
Compare
Changes
isMinimal
state to control different UIScreen.Recording.2025-01-15.at.18.02.16.mov