fix: show sort direction in filter bar in trending list#29809
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. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 96e8ebb. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
The Trending feature (discovery tab with Tokens section) is covered by SmokeWalletPlatform. No other tags are needed because:
No dependent tags are required per the tag descriptions since this is a Trending UI-only change. Performance Test Selection: |
|




Description
Add sort direction indicators to trending token filter buttons to improve UX by making the current sort order immediately visible.
Changelog
CHANGELOG entry: Adds sort icons on filter bar in trending list
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/ASSETS-3152
Manual testing steps
Screenshots/Recordings
Before
After
Screen.Recording.2026-05-07.at.14.44.42.mov
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
Low risk UI enhancement that only adds derived state and icon rendering for the sort button; minimal chance of regressions outside the Trending filter bar layout.
Overview
Adds a sort-direction indicator to the Trending token list price-change filter.
FilterButton/FilterBarnow accept an optionaliconName/priceChangeIconNameand render that icon before the label.useTokenListFiltersexposes a newpriceChangeSortDirectionIcon(up/down) derived frompriceChangeSortDirection,TokenListPageLayoutwires it into theFilterBar, and tests cover the new icon mapping.Reviewed by Cursor Bugbot for commit 0c6e926. Bugbot is set up for automated code reviews on this repo. Configure here.