fix: standardize error display to use showError/hideError helpers #90
Closed
clawbot
wants to merge 1 commits from
fix/87-consistent-error-display into main
pull from: fix/87-consistent-error-display
merge into: :main
:main
:issue-398-fresh-provider-uuid
:issue-369-approval-amount-vocabulary
:next
:feat/issue-131-transaction-view-layout
:fix/116-timestamps-include-timezone
:feature/expanded-warnings
:fix/cross-wallet-duplicate-detection
:feature/copy-flash-feedback
:feature/issue-82-new-address-warning
:feature/82-warn-new-address
:issue-99-block-number-external-link
:issue-99-block-number-styling
:feature/82-zero-history-warning
:fix/issue-99-block-number-link-copy
:fix/99-block-number-clickable
:fix/87-consistent-error-display-v2
:fix/87-consistent-error-display
:fix/consistent-error-display
:fix/77-confirm-tx-persist
:fix/issue-72-address-token-tx-history
:fix/70-confirm-tx-contract-display
:fix/issue-58-receive-address-consistency
:fix/55-swap-show-own-address
:feature/show-private-key
1
Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
|
5e4cf34ef8 |
fix: standardize error display to use showError/hideError helpers
check / check (push) Successful in 9s
Replace three inconsistent error display patterns with the centralized showError()/hideError() helpers from helpers.js: - approval.js: replace direct DOM classList toggling for approve-tx-error and approve-sign-error with showError/hideError calls - addressDetail.js: replace export-privkey-flash direct DOM with showError/hideError using renamed export-privkey-error element - deleteWallet.js: replace delete-wallet-flash direct DOM with showError/hideError using renamed delete-wallet-error element - addWallet.js: replace showFlash() validation errors with dedicated add-wallet-error div and showError/hideError calls - importKey.js: replace showFlash() validation errors with dedicated import-key-error div and showError/hideError calls - index.html: add error divs for add-wallet and import-key views, rename export-privkey-flash to export-privkey-error, rename delete-wallet-flash to delete-wallet-error, remove inconsistent text-red-500 class Closes #87 |