Skip to content

Feature/trde guide#6227

Draft
SeniorZhai wants to merge 2 commits intofeature/trading_perpetualfrom
feature/trde_guide
Draft

Feature/trde guide#6227
SeniorZhai wants to merge 2 commits intofeature/trading_perpetualfrom
feature/trde_guide

Conversation

@SeniorZhai
Copy link
Member

No description provided.

@SeniorZhai SeniorZhai added the testing Now testing, but you can review label Mar 17, 2026
@SeniorZhai
Copy link
Member Author

@claude[agent]

Target files

  • ./app/src/main/java/one/mixin/android/ui/home/web3/trade/CandleChart.kt
  • ./app/src/main/java/one/mixin/android/ui/home/web3/trade/perps/PerpsMarketDetailPage.kt

1. Candle zoom behavior

  • Default zoom must be 1x.
  • Minimum zoom must be 1x (do not allow zooming out below default).
  • Maximum zoom must be 3x.
  • Zoom must affect only candle body width (single candle width), not spacing.
  • Two-finger pinch must feel responsive.
  • During two-finger pinch, do not trigger single-finger move/drag/selection behaviors.

2. Delayed candle detail on press (down)

  • On pointer down, do not show candle detail immediately.
  • Add a small delay before showing detail (about 100-150ms).
  • If user releases, moves, scrolls, or starts pinch during that delay, cancel showing detail.
  • When detail is shown, display:
    • Open
    • High
    • Low
    • Close
    • Volume

3. Header layout above CandleChart

Header must keep a fixed total height across states.

Default state

Must render exactly as:

  • Title
  • Price + ICON on the far right (48dp)
  • Change

Candle-detail state

Must render exactly as:

  • Title + ICON on the far right (20dp)
  • Open High Low Close Volume

Additional layout constraints:

  • Price/Change must be hidden in candle-detail state.
  • Icon size transition can be animated, but must not change parent container height.
  • Icon container must not overlap/crop OHLCV content.

4. Keep existing behavior intact

  • Keep existing loading/error/empty states.
  • Keep existing candle drawing behavior.
  • Keep existing timeframe switch behavior.
  • Keep existing fiat conversion behavior for right-axis/price labels (do not regress).

@Claude
Copy link
Contributor

Claude AI commented Mar 17, 2026

@SeniorZhai I've opened a new pull request, #6228, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Now testing, but you can review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants