fix: if should not be always true

[cuiweixie]

Description

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Low Risk
Low risk UI-only change that fixes an always-truthy conditional affecting avatar overlap spacing; may slightly change layout for some avatar sizes.

Overview
Fixes AvatarGroup’s overlap spacing logic by comparing size against AvatarTokenSize enums (instead of using always-truthy constants), so marginLeftValue is computed correctly for Xs/Sm/default sizes.

^{Written by Cursor Bugbot for commit 96371b9. This will update automatically on new commits. Configure here.}

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[cuiweixie]

I have read the CLA Document and I hereby sign the CLA

[HowardBraham]

@cuiweixie I think this is correct, but can you find an actual bug in the client that this fixes?

[cuiweixie]

I have try.But it's hard to find px diff by eye without some tool.

[cuiweixie]

It seems the ci failed is unrelated to the pr?