Skip to content

feat: ✨ update group meeting cards#564

Open
Windslash123 wants to merge 4 commits into
mainfrom
aa/update-meeting-cards
Open

feat: ✨ update group meeting cards#564
Windslash123 wants to merge 4 commits into
mainfrom
aa/update-meeting-cards

Conversation

@Windslash123

@Windslash123 Windslash123 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Description

Recording/Screenshots

Before

image

After

image

Test Plan

Issues

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

3 issues found across 3 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/components/ui/meeting-card.tsx Outdated
Comment thread src/components/groups/group-member-list.tsx
Comment thread src/components/groups/group-member-list.tsx
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@ethancha0

Copy link
Copy Markdown
Collaborator
image

Looks great on mobile!

@ethancha0

Copy link
Copy Markdown
Collaborator
image image

issue: responder count and responder total numbers should be swapped

@ethancha0 ethancha0 self-requested a review June 5, 2026 16:20

@ethancha0 ethancha0 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Love the new consistent design, just need to swap the responder fields to accurately display the number of responders in a meeting

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.

mobile group card overflow

2 participants