Skip to content

Fix member count position on Project Dashboard cards#583

Open
nshan20 wants to merge 1 commit into3.0from
3.0-535
Open

Fix member count position on Project Dashboard cards#583
nshan20 wants to merge 1 commit into3.0from
3.0-535

Conversation

@nshan20
Copy link
Collaborator

@nshan20 nshan20 commented Mar 10, 2026

No description provided.

@aamialkovich
Copy link
Collaborator

Please resolve conflicts before merge

@if (variant === 'grid') {
<div class="flex items-center">
<div class="flex -space-x-2">
@for (member of project.acl.slice(0, 3); track member) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

acl for list requests will be limited to 3 items, so no need to slice

<div
class="flex items-center justify-center w-[1.6rem] h-[1.6rem] rounded-full bg-[#e0e0e0] text-[#646464] text-[0.625rem] font-semibold uppercase border border-white"
>
{{ member.username ?? member.displayName | initials }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

If it doesn't contradict specs, please use displayName first and use username if displayName is not available, seems to be user-friendly (especially with initials)

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.

3 participants