feat(ui): Show full pubkeys where copyable#2955
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Visit the preview URL for this PR (updated for commit 693f5dd): https://walletrc--pull-2955-merge-tsi0hco5.web.app (expires Tue, 29 Jul 2025 11:32:08 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc |
|
@smk762 It's been given the thumbs up from Poly |
* feat(ui): auto-scroll addresses in tx pages * feat: show full transaction address * feat: show full address for own addresses * chore: roll SDK * chore: roll SDK * fix(ui): Fix address text item responsiveness * fix(ui): align transaction table columns
* Adds `invalidWalletFileNameError` locale text * disallow wallet import if filename has illeagal chars * adds local message for `invalidWalletNameError` * dissallow seed import or wallet creation if wallet name has illegal chars * block creating wallet name with illegal chars * rm dupe declaration * trim wallet name before validation * Update lib/views/wallets_manager/widgets/wallet_creation.dart Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * q pdate "seed" to "seed phrase" in login/import forms (#2972) * fix translation string name * `seed` - > `seed phrase` * removes "seed" in connect button * feat(ui): Show full pubkeys where copyable (#2955) * feat(ui): auto-scroll addresses in tx pages * feat: show full transaction address * feat: show full address for own addresses * chore: roll SDK * chore: roll SDK * fix(ui): Fix address text item responsiveness * fix(ui): align transaction table columns * feat(settings): hide log export when logged out (#2967) * bugfix: disable withdraw preview if source address not selected (#2969) * remove duplicate transaltion key/value * fix source address unselected validation bug * fix(ui): avoid autofocus conflicts (#2968) * fix(ui): remove duplicate CircularProgressIndicator on address creation (#2970) --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
* Adds `invalidWalletFileNameError` locale text * disallow wallet import if filename has illeagal chars * adds local message for `invalidWalletNameError` * dissallow seed import or wallet creation if wallet name has illegal chars * block creating wallet name with illegal chars * rm dupe declaration * trim wallet name before validation * Update lib/views/wallets_manager/widgets/wallet_creation.dart Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * q pdate "seed" to "seed phrase" in login/import forms (#2972) * fix translation string name * `seed` - > `seed phrase` * removes "seed" in connect button * feat(ui): Show full pubkeys where copyable (#2955) * feat(ui): auto-scroll addresses in tx pages * feat: show full transaction address * feat: show full address for own addresses * chore: roll SDK * chore: roll SDK * fix(ui): Fix address text item responsiveness * fix(ui): align transaction table columns * feat(settings): hide log export when logged out (#2967) * bugfix: disable withdraw preview if source address not selected (#2969) * remove duplicate transaltion key/value * fix source address unselected validation bug * fix(ui): avoid autofocus conflicts (#2968) * fix(ui): remove duplicate CircularProgressIndicator on address creation (#2970) --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
This pull request introduces several changes across different parts of the codebase, focusing on improving text presentation with auto-scrolling functionality, simplifying UI layouts, and enhancing code maintainability. The key updates include replacing static
Textwidgets withAutoScrollTextfor better text visibility in constrained spaces, removing unused imports, and refining layout structures.Text Presentation Enhancements:
Textwidgets withAutoScrollTextinCopiedTextandCopiedTextV2classes to enable auto-scrolling for long text values. (lib/shared/widgets/copied_text.dart, [1] [2]AddressTextto support both truncated and auto-scrolling text based on theisTruncatedflag. (lib/views/wallet/common/address_text.dart, lib/views/wallet/common/address_text.dartR2-R26)Layout Refinements:
FlexibleandExpandedwidgets in various UI components, such asAddressCardandTransactionDetails, to improve layout flexibility and responsiveness. (lib/views/wallet/coin_details/coin_addresses.dart, [1];lib/views/wallet/coin_details/transactions/transaction_details.dart, [2]_Listlayout intransaction_list.dartby removing unnecessaryCardwrapping. (lib/views/wallet/coin_details/transactions/transaction_list.dart, lib/views/wallet/coin_details/transactions/transaction_list.dartL54-R55)Code Cleanup:
missing_coins_dialog.dartto reduce clutter and improve maintainability. (lib/views/support/missing_coins_dialog.dart, lib/views/support/missing_coins_dialog.dartL3-L16)Consistency Improvements:
HistoryTitletext to use a more generic label, changing "Last Transactions" to "Transactions." (lib/views/wallet/coin_details/transactions/transaction_list.dart, lib/views/wallet/coin_details/transactions/transaction_list.dartL147-R140)Additional Changes:
AutoScrollTextin transaction-related components, such asTransactionListRowand_TransactionAddress, for consistent text scrolling behavior. (lib/views/wallet/coin_details/transactions/transaction_list_item.dart, [1] [2]