Skip to content

Conversation

@sktbrd
Copy link
Contributor

@sktbrd sktbrd commented Dec 14, 2025

This pull request introduces several UI and UX improvements across the KeepKey browser extension, focusing on enhanced token/asset management, improved branding consistency, and clearer transaction information. The most significant updates include a redesigned side panel with total balance display and global send/receive actions, improved handling of project logos and favicons, and clearer transaction risk indicators.

User Interface & Experience Improvements

  • The side panel (SidePanel.tsx) now displays the user's total USD balance and provides global "Send" and "Receive" buttons, making asset management more intuitive. When these actions are triggered, the asset with the highest USD value is selected by default, and new full-screen drawers are used for sending and receiving assets. The panel header has also been streamlined with improved status indicators and tooltips. [1] [2] [3] [4]
  • The Balances component introduces chain name mapping for cleaner display and adds new UI elements for improved asset selection and navigation.

Branding & Logo Consistency

  • All instances of the KeepKey logo in project info cards have been updated to use a consistent URL (https://pioneers.dev/coins/keepkey.png) and improved fallback handling. The EVM ProjectInfoCard now distinguishes between dApps and the KeepKey extension, using a square logo for KeepKey and a round avatar for dApps, with robust error handling for logo loading. [1] [2] [3] [4]

Transaction & Risk Clarity

  • The RequestMethodCard now uses clearer iconography and colors to indicate transaction safety/risk. Simple transfers are shown as safe (green, checkmark), while smart contract interactions are highlighted as requiring caution (yellow, warning icon). Descriptions have been updated for clarity.

Wallet Support Enhancements

  • The KeepKey background script now supports additional Ethereum accounts (account 1 and 2) by adding their derivation paths, enabling users to access multiple Ethereum addresses.

Codebase Cleanup

  • Removed an unused import (AssetValue) from the KeepKey background script for minor codebase hygiene.

* Please fill in the required items.

Priority*

  • High: This PR needs to be merged first for other tasks.
  • Middle: This PR should be merged quickly to prevent conflicts due to common changes. (default)
  • Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*

Ui polish

Changes*

Descirbed by copilot above

- Token-first wallet view with unified token list and chain badges
- Global Send/Receive buttons with total balance display
- Redesigned Send modal with clean inputs, 50%/Max buttons, USD toggle
- Redesigned Receive modal with QR code + token logo, token selector, account selector
- Added Ethereum accounts 1 & 2 derivation paths (m/44'/60'/1'/0/0, m/44'/60'/2'/0/0)
- Fixed KeepKey logo in popup transaction screens
- Changed transfer safety indicator to green checkmark for simple transfers
- Improved drawer navigation with back arrow instead of X close button
- Deduplicated chain selector to show unique tokens only
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution. We will check and reply to you as soon as possible.

@BitHighlander BitHighlander merged commit 0fe6330 into keepkey:master Dec 16, 2025
1 check passed
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.

2 participants