Skip to content

feat: add fee tier popover to perp trading panel#10281

Draft
rayston92 wants to merge 1 commit intoxfrom
perp-fees
Draft

feat: add fee tier popover to perp trading panel#10281
rayston92 wants to merge 1 commit intoxfrom
perp-fees

Conversation

@rayston92
Copy link
Copy Markdown
Collaborator

@rayston92 rayston92 commented Feb 17, 2026 •
edited by devin-ai-integration Bot
Loading

Summary

  • Add a "Fee Tier" popover below trading buttons in the perp order panel
  • Your Fees tab: shows Builder Fee (OneKey: 0%), Hyperliquid Taker/Maker fees with staking discount, total fees, and Fee Tier / Staking Tier badges
  • Compare tab: 7-wallet comparison (OneKey, Dreamcash, Phantom, Infinex, Liquid, Rainbow, MetaMask) with icons, brand-color progress bars, and "Overall Best" badge on OneKey
  • Demo hardcoded data (Fee Tier 3 / Gold Staking) for now, to be replaced by Hyperliquid userFees API later

Test plan

  • Open Perp trading panel on web, verify "% Fee Tier" button appears below trading buttons (left-aligned)
  • Click to open popover — should open upward, not overflow screen
  • Verify "Your Fees" tab shows correct fee breakdown and tier badges
  • Switch to "Compare" tab, verify wallet list with progress bars and "Overall Best" badge on OneKey
  • Test on mobile — popover should render as bottom sheet

🤖 Generated with Claude Code

Open with Devin

@rayston92
feat: add fee tier popover to perp trading panel
9077059 
Add a "Fee Tier" popover below the trading buttons in the perp order
panel, showing fee breakdown and competitive wallet comparison.

- Your Fees tab: Builder Fee, Hyperliquid Taker/Maker fees with
  staking discount, total fees, and tier/staking badges
- Compare tab: 7-wallet comparison with icons, progress bars in
  brand colors, and "Overall Best" badge on OneKey
- Popover opens upward to avoid screen overflow
- Demo hardcoded data (Fee Tier 3 / Gold Staking) for now,
  to be replaced by Hyperliquid userFees API later
devin-ai-integration[bot]
devin-ai-integration Bot reviewed Feb 17, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

Open in Devin Review

@revan-zhang
Copy link
Copy Markdown
Contributor

revan-zhang commented Feb 17, 2026 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@originalix originalix marked this pull request as draft February 24, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@ezailWang ezailWang Awaiting requested review from ezailWang ezailWang is a code owner

@revan-zhang revan-zhang Awaiting requested review from revan-zhang revan-zhang is a code owner

@sidmorizon sidmorizon Awaiting requested review from sidmorizon sidmorizon is a code owner

@originalix originalix Awaiting requested review from originalix originalix is a code owner

@huhuanming huhuanming Awaiting requested review from huhuanming huhuanming is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rayston92 @revan-zhang