-
Notifications
You must be signed in to change notification settings - Fork 528
feat: add market list filters and client-side column sorting #12898
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
erikzou
wants to merge
86
commits into
OneKeyHQ:x
Choose a base branch
from
erikzou:feat/market-list-sort-filter
base: x
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 76 commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
e72108f
feat: add market list client sort util with missing-value sink
erikzou 2c305ef
feat: open all columns for market list client sort
erikzou b41ccd4
feat: enable full-column header sort on trending tab
erikzou 3961680
fix: avoid nested ternary in client sort key resolution
erikzou 47822a2
feat: hide injected BTC row behind handoff switch
erikzou 8f84d49
feat: add showMarketListRedesign dev flag
erikzou 8abbc00
feat: add market list filter state model and local adapter
erikzou a021c62
feat: add market filter chips bar with tier popover
erikzou 9b88832
feat: add market filters popover with tier rows
erikzou 8a4ca91
fix: wrap filters badge count in Badge.Text and slugify audit testIDs
erikzou 4d91cd8
feat: wire filter chips demo into desktop trending tab
erikzou 20237a8
feat: add redesigned trending columns with reason tag placeholder
erikzou 3e19ea9
fix: gate market list redesign to trending tab only
erikzou 51f5506
feat: add dex filter chip analytics
erikzou e5844d7
fix: use toSorted in market list client sort
erikzou 42f9031
fix: constrain market filters popover height and wrap tier rows
erikzou 3568aaa
fix: reduce filters popover scroll area for short windows
erikzou e21e10a
fix: keep BTC-hidden trending list from re-opening pagination
erikzou 8ccb7b1
fix: restrict banner detail client sort to change24h column
erikzou 64b5c00
feat: align market list redesign UI with Figma specs
erikzou 9e185a7
fix: stabilize filter chips bar height and unify tier pill style
erikzou d917517
feat: rework market filter tiers into dimension buckets
erikzou 8e1e91f
feat: align filters popover tier grid and scrolling tabs
erikzou eea97e1
fix: uniform fixed-width tier pills and drop columnless filter dimens…
erikzou 6a4c522
feat: fit four tier pills per row and unit-suffixed row titles
erikzou 9200b54
feat: replace filters popover with grouped modal
erikzou b87b71d
feat: refine redesign table header and first column metrics
erikzou e9b7bbd
fix: use size token for header sort icon and restore price sorting
erikzou f8cca38
fix: set redesign row height to 68px and pad sticky header
erikzou b65ae51
fix: narrow redesign name column to 208px
erikzou f505bf1
feat: add copy affordance to redesign address cell
erikzou 1ad60f4
fix: recolor address copy target on hover instead of fading
erikzou a4b927f
feat: lay out filter rows with left labels and metrics/audit groups
erikzou 7d0edd0
feat: restyle filters modal to match design spec
erikzou 2c0b11f
refactor: order filter rows by stock then flow metrics
erikzou 04c5a98
fix: drop non-reactive time frame row from filters modal
erikzou f222811
fix: restore time frame row with draft-applied selection
erikzou 63b82d9
fix: keep filter row labels on a single line
erikzou 402e1f9
fix: name audit group after the metric its rows measure
erikzou 9632a16
feat: unify chip and header sort state, add chip tooltips
erikzou ed64b49
feat: rebuild P2-9 chips on the frozen 2026-07-21 spec
erikzou 8a38f52
feat: no-jitter hover remove on condition chips, trim tier rows
erikzou 8ac7b30
feat: condition chip close-button hover recolor, reuse GradientMask
erikzou 6a8e534
fix: opaque chip fade plateau, inert sort-chip body
erikzou 1357592
feat: single-row tier popover, sort chip dropdown with disabled asc
erikzou 23c8097
fix: widen tier popover so three floor labels fit one row
erikzou 96872be
style: tier popover title to bodySmMedium
erikzou 26712e0
feat: fade-slide transition on chip <-> pill swap
erikzou ee59b15
style: condition-chip and filter-label text to text/text
erikzou 96325f7
style: Filters badge to info color
erikzou 7cb238d
feat: move trending filters into the bordered sub-category toolbar
erikzou 641b7c4
style: rebalance toolbar spacing 8px up
erikzou 3ddc3d9
refactor: one shared toolbar frame across all four Market tabs
erikzou 732997b
style: trending toolbar buttons take the category pill shape
erikzou 86f774b
fix: play the chip transition on expand, not on load
erikzou 4f5cf86
chore: hide the mock reason tag for this delivery
erikzou 4ae5b46
chore: drop two dev-note captions from the Filters modal
erikzou 50d3e20
feat: add the Filters entry to the mobile toolbar
erikzou c0e4edc
Merge remote-tracking branch 'upstream/x' into feat/market-list-sort-…
erikzou bb7da24
fix: carry the Filters entry into the native mobile layout
erikzou 3eef0cf
feat: hide the holdings audit filter group for this wave
erikzou 87dc202
feat: pass market filter conditions through to the hot-token API
erikzou fdb4edf
fix: drop the unsupported tabPressAnimationEnabled prop
erikzou df9c397
test: lock token age as the only client-side filter dimension
erikzou 14b17ae
feat: share the redesigned list visuals and client sort across market…
erikzou 60c0b39
feat: let every banner detail column sort on the client
erikzou e319f5e
fix: keep ascending selectable for a hand-driven sort
erikzou a6cba4a
feat: localize the market filter panel, chips and header tooltips
erikzou 44a4d84
fix: fit the token age tiers in the filter pill
erikzou 5c5c819
fix: align the perps mobile rows with the spot lists
erikzou 48d8d9d
fix: name the market volume columns by the window they report
erikzou 062315b
fix: give the perps desktop table the shared row geometry
erikzou 1564eba
Merge remote-tracking branch 'upstream/x' into feat/market-list-sort-…
erikzou d6cac55
fix: restore tabPressAnimationEnabled now that Tabs declares it
erikzou 7641a1e
refactor: ship the market list redesign without the dev flag
erikzou 7259124
fix: address review findings on market sort, filters and analytics
erikzou e6c3df5
chore: pull translations after token age wording fix
erikzou 3f79d1d
docs: record the measured pagination behavior behind client sort
erikzou 0068eae
refactor: gate the market filters entry on the list that consumes it
erikzou adc8101
revert: stop hiding the injected BTC row in trending
erikzou 49df56e
fix: keep the sort on an unchanged Confirm and stop double-firing hea…
erikzou b5e2b14
Merge remote-tracking branch 'upstream/x' into feat/market-list-sort-…
erikzou bc30da4
fix: drop watchlist header sorting and sort banner perps
erikzou 91c3679
fix: keep the sort on an unchanged filter and anchor presets to their…
erikzou b5007aa
style: wrap the preset time-range guard
erikzou eda5ecd
feat: apply quick filter chips on the user's selected time frame
erikzou File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
在移动端任意具有时间范围选择器的非股票 Spot 标签中,这里都会显示筛选按钮,但
MobileMarketTokenFlatList只有在selectedCategory === 'trending'时才把条件传给请求或本地过滤。因此用户在 AI、Meme 等普通分类中打开弹窗并确认后,徽标会更新而列表完全不变。请把当前分类传入并隐藏或禁用非 Trending 标签中的入口。Useful? React with 👍 / 👎.