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.

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

feat: update legal links display and display external wallet name #26

Merged
merged 5 commits into from
Jan 28, 2025

Conversation

jeremy0x
Copy link
Member

@jeremy0x jeremy0x commented Jan 27, 2025

This pull request includes several changes across multiple files to enhance the functionality and user experience of the application. The main changes involve adding new components, updating existing components, and improving the handling of dropdowns.

New Features:

  • Added NoblocksBetaLogo component to app/components/ImageAssets.tsx to provide a new logo option.

Enhancements to Existing Components:

  • Updated Navbar component in app/components/Navbar.tsx to include a new dropdown menu that closes when clicking outside of it. This involved adding a useRef hook and a handleClickOutside function. [1] [2] [3]
  • Simplified currency display in PDFReceipt component by removing the formatCurrency function and directly displaying the amount and currency.

Codebase Improvements:

  • Added useWallets import and integrated it into the WalletDetails component to handle external wallets more effectively. [1] [2] [3]

Minor Adjustments:

  • Updated TransferModal component to improve accessibility and focus styles.
  • Added new environment variables for analytics in .env.example.

Screenshots

image

image

@jeremy0x jeremy0x added the enhancement New feature or request label Jan 27, 2025
@jeremy0x jeremy0x self-assigned this Jan 27, 2025
app/components/ImageAssets.tsx Show resolved Hide resolved
app/components/Navbar.tsx Outdated Show resolved Hide resolved
@jeremy0x jeremy0x changed the title feat: add NoblocksBetaLogo component and update links style feat: update legal links display and display external wallet name Jan 27, 2025
@chibie chibie merged commit 8247355 into main Jan 28, 2025
@chibie chibie deleted the fix/navbar branch January 28, 2025 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants