Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore(tangle-dapp): Selector Modals, Bridge UI cleanup & Remove Sygma Bridge #2596

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from
Draft

chore(tangle-dapp): Selector Modals, Bridge UI cleanup & Remove Sygma Bridge #2596

wants to merge 3 commits into from

Conversation

devpavan04
Copy link
Member

@devpavan04 devpavan04 commented Oct 22, 2024 •
edited
Loading

Summary of changes

  • ➕ Adds reusable selector modals (Chain/Token/Operator) in tangle-dapp/components.
  • 🧹 Bridge UI fixes: Corrects color, border, and padding discrepancies. Adds missing labels and general UI cleanup.
  • ❌ Removes Sygma bridge. The bridge now only displays Hyperlane-supported chains and tokens.

Proposed area of change

  • apps/tangle-dapp
  • libs/webb-ui-components

Associated issue(s)

Screenshots and Screen Recordings

CleanShot.2024-10-22.at.05.54.03.mp4

CleanShot 2024-10-22 at 05 51 30@2x

CleanShot 2024-10-22 at 05 55 14@2x

@devpavan04 devpavan04 self-assigned this Oct 22, 2024
@netlify Netlify
Copy link

netlify bot commented Oct 22, 2024 •
edited
Loading

✅ Deploy Preview for tangle-dapp ready!

Name Link
🔨 Latest commit 442635c
🔍 Latest deploy log https://app.netlify.com/sites/tangle-dapp/deploys/6717a0f5f52fc600082f86ed
😎 Deploy Preview https://deploy-preview-2596--tangle-dapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@devpavan04 devpavan04 added the tangle-dapp Tangle Staking dApp label Oct 22, 2024
@drewstone
Copy link
Contributor

Some merge conflicts here. @devpavan04 I would suggest showing also more characters in the addresses than we are currently.

@devpavan04 devpavan04 marked this pull request as draft October 22, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

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

Assignees

@devpavan04 devpavan04

Labels
tangle-dapp Tangle Staking dApp
Projects
None yet
Milestone
No milestone
2 participants
@devpavan04 @drewstone