Skip to content

fix(defi): Pendle PT convergence chart redesign and Earn UI fixes (OK-51132)#10534

Merged
originalix merged 17 commits intoOneKeyHQ:xfrom
erikzou:fix/pendle
Mar 10, 2026
Merged

fix(defi): Pendle PT convergence chart redesign and Earn UI fixes (OK-51132)#10534
originalix merged 17 commits intoOneKeyHQ:xfrom
erikzou:fix/pendle

Conversation

@erikzou
Copy link
Copy Markdown
Contributor

@erikzou erikzou commented Mar 9, 2026 •
edited by devin-ai-integration bot
Loading

Summary

  • Redesign Pendle PT convergence chart with wavy dashed curve, gradient fill, and dynamic curve scaling based on rate gap
  • Improve chart RWD: separate SVG graphics from HTML text overlays using flex layout, so the chart scales horizontally while maintaining height on small screens (80% height on mobile)
  • Show remaining days to maturity on chart x-axis and align target label with curve end
  • Add font-weight 600 to PT symbol price label in convergence chart
  • Rename first "All assets" tab label from "DeFi" to "Simple Earn"

Related Issues

Changed files

File Changes
PendlePtConvergenceChart.tsx Restructure to flex layout (SVG chart + HTML text overlays) with preserveAspectRatio="none" and responsive height via useMedia
PendlePtConvergenceChart.native.tsx Update native WebView chart with matching curve and label changes
shared.ts Add shared constants, curve paths, and utility functions for chart scaling
AvailableAssetsTabViewList.tsx Rename tab label from "DeFi" to "Simple Earn"
CleanShot 2026-03-06 at 18 25 09@2x
Open with Devin

erikzou added 5 commits March 6, 2026 00:55
fix(defi): redesign Pendle PT convergence chart with wavy dashed curve
c246958
fix(defi): show remaining days to maturity and align target label wit…
c6ca00c 
…h curve end
fix(defi): rename first All assets tab label from DeFi to Simple Earn
0f259ba
fix(defi): add font-weight 600 to PT symbol price label in convergenc…
ade9f28 
…e chart
fix(defi): improve RWD for Pendle PT convergence chart on small screens
4116db1 
Restructure chart layout to separate SVG graphics from HTML text overlays
using a flex row (chart area + fixed-width labels column). The SVG uses
preserveAspectRatio="none" so the chart scales horizontally while
maintaining height. Text labels and dot are HTML overlays positioned with
percentage-based coordinates to stay crisp and unscaled. On small screens
(<=md), chart height reduces to 80% of full size.
@revan-zhang
Copy link
Copy Markdown
Contributor

revan-zhang commented Mar 9, 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.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

Sign in to view

devin-ai-integration[bot]
devin-ai-integration bot reviewed Mar 9, 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

fix(defi): use fixed curve shape in Pendle PT convergence chart
eb2a0d1 
The chart is an illustrative diagram — the curve no longer scales
dynamically with the price gap. Only the right-side labels update
to reflect actual current/target rates. This prevents the curve from
being clipped at the bottom when the price gap is large.
chatgpt-codex-connector[bot]

This comment was marked as resolved.

Sign in to view

devin-ai-integration[bot]

This comment was marked as resolved.

Sign in to view

@zhaono1 zhaono1 enabled auto-merge (squash) March 9, 2026 04:05
@originalix originalix disabled auto-merge March 9, 2026 07:24
devin-ai-integration[bot]

This comment was marked as resolved.

Sign in to view

chatgpt-codex-connector[bot]
chatgpt-codex-connector bot reviewed Mar 9, 2026
View reviewed changes
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c8f1820c09

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ...ews/Staking/components/ProtocolDetails/PendlePtConvergenceChart/PendlePtConvergenceChart.tsx
@limichange limichange enabled auto-merge (squash) March 10, 2026 07:18
@originalix originalix disabled auto-merge March 10, 2026 07:32
@originalix originalix merged commit 3f554b3 into OneKeyHQ:x Mar 10, 2026
7 checks passed
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

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

@originalix originalix originalix approved these changes

@zhaono1 zhaono1 zhaono1 approved these changes

@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

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

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@erikzou @revan-zhang @originalix @zhaono1