Skip to content

fix: chart#24

Merged
priom merged 2 commits intoprodfrom
minor-ui-updates
Apr 30, 2026
Merged

fix: chart#24
priom merged 2 commits intoprodfrom
minor-ui-updates

Conversation

@priom
Copy link
Copy Markdown
Member

@priom priom commented Apr 30, 2026

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the TradingView chart integration by enabling the volume indicator and setting visible plots to 'ohlcv'. It also rebrands 'Connect Wallet' to 'Login' across the UI, renames the 'Bridge' button to 'Deposit', and comments out the legacy DepositModal. Configuration changes include adding Polygon to the bridge modal while reducing the default max leverage and market order slippage. Feedback highlights a typo in the mobile logout button's aria-label and suggests removing dead code in global-modals.tsx. Additionally, concerns were raised regarding the significant reduction of popular symbols in the bridge modal and the drastic decrease in default max leverage.

Comment thread src/components/trade/tradebox/across-bridge-modal.tsx
Comment thread src/config/constants.ts
Comment thread src/components/trade/mobile/mobile-account-view.tsx Outdated
Comment thread src/components/trade/components/global-modals.tsx
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 30, 2026

Deploying hyperterminal-mainnet with  Cloudflare Pages  Cloudflare Pages

Latest commit: b447392
Status:⚡️  Build in progress...

View logs

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying hyperterminal with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5fcb923
Status: ✅  Deploy successful!
Preview URL: https://4fb046d9.hyperterminal.pages.dev
Branch Preview URL: https://minor-ui-updates.hyperterminal.pages.dev

View logs

@priom priom merged commit f6a28d9 into prod Apr 30, 2026
3 of 4 checks passed
@priom priom deleted the minor-ui-updates branch April 30, 2026 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant