fix: standardize error display to use showError/hideError helpers (closes #87) #96
Closed
clawbot
wants to merge 1 commits from
fix/87-consistent-error-display-v2 into main
pull from: fix/87-consistent-error-display-v2
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 | |
|---|---|---|---|---|
|
|
db1b5c1c69 |
fix: standardize error display to use showError/hideError helpers
check / check (push) Successful in 22s
Replace inconsistent error display patterns across views:
- addWallet.js: replace showFlash() with showError('add-wallet-error', ...)
- importKey.js: replace showFlash() with showError('import-key-error', ...)
- addressDetail.js: replace direct DOM manipulation of export-privkey-flash
with showError/hideError('export-privkey-error', ...)
- deleteWallet.js: replace direct DOM manipulation of delete-wallet-flash
with showError/hideError('delete-wallet-error', ...)
- index.html: add dedicated error divs with min-h-[1.25rem] for add-wallet,
import-key views; rename flash divs to error divs for export-privkey and
delete-wallet views with consistent styling
All password/validation errors now use the showError()/hideError() helper
pattern with min-h-[1.25rem] error divs to prevent layout shift.
Status/success messages (scanning, wallet deleted) remain as showFlash().
Closes #87
|