chore: polish trader profile visual nits#32388
Conversation
|
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. |
PR template — items to address before "Ready for review"Warnings — informational, address before merging:
See docs/readme/ready-for-review.md for the full Definition of Ready for Review. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
These are UI display changes within the Social Leaderboard / Perps feature area. SmokePerps is selected because the changes directly affect perp position display components (PerpBadges, PositionRow) used in the trader profile. SmokeWalletPlatform is selected per the tag description which states Perps is a section inside the Trending tab and changes to Perps views affect Trending. SmokeConfirmations is NOT selected because these changes don't touch the Add Funds flow or any transaction confirmation path — they only affect display formatting in the leaderboard/position list views. No core controllers, navigation, shared modals, or infrastructure are affected. Performance Test Selection: |
|



Description
Fix Open/Closed tab underline to use icon-default and align tab labels by reserving underline space on both tabs
Align perp badges with the rest of social leaderboard: title-case Long/Short labels, matched leverage/direction pill sizing, and signed +/- PnL percents instead of triangles
Changelog
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/TSA-812
Manual testing steps
Screenshots/Recordings
Before
NA
After
NA
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Low Risk
Presentation-only changes in Social Leaderboard UI with no auth, data, or API impact; risk is limited to display regressions caught by updated tests.
Overview
Polishes Social Leaderboard trader profile visuals so tabs, position PnL, and perp metadata read consistently.
Open/Closed tabs on
TraderProfileViewnow use a full-widthicon-defaultunderline on the active tab, with a transparent bar on the inactive tab so labels stay aligned.Position rows drop ▲/▼ and unsigned percentages in favor of signed PnL percents (e.g.
+182.00%,-25.00%,+0.00%at break-even) via defaultformatPercentinPositionRow.Perp badges show localized Long/Short (no forced uppercase), and leverage uses the same muted pill styling as direction instead of
PerpsLeverage; unit tests were updated to match.Reviewed by Cursor Bugbot for commit 36b690b. Bugbot is set up for automated code reviews on this repo. Configure here.