sneak/AutistMask
1
0
Fork 0
Code Issues 66 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

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
Conversation 1 Commits 1 Files Changed 6
+55 -46
clawbot commented 2026-02-28 22:07:01 +01:00
Collaborator
Copy Link
Copy Source

Replace three inconsistent error display patterns with the centralized showError()/hideError() helpers from helpers.js.

Changes:

  • approval.js: replace direct DOM classList toggling with showError/hideError
  • addressDetail.js: replace export-privkey-flash direct DOM with showError/hideError (element renamed to export-privkey-error)
  • deleteWallet.js: replace delete-wallet-flash direct DOM with showError/hideError (element renamed to delete-wallet-error, removed inconsistent text-red-500)
  • addWallet.js: replace showFlash() validation errors with dedicated add-wallet-error div
  • importKey.js: replace showFlash() validation errors with dedicated import-key-error div
  • index.html: add error divs for add-wallet and import-key views, rename flash elements to error elements

docker build . passes.

Closes #87

Replace three inconsistent error display patterns with the centralized `showError()`/`hideError()` helpers from `helpers.js`. **Changes:** - `approval.js`: replace direct DOM classList toggling with `showError`/`hideError` - `addressDetail.js`: replace `export-privkey-flash` direct DOM with `showError`/`hideError` (element renamed to `export-privkey-error`) - `deleteWallet.js`: replace `delete-wallet-flash` direct DOM with `showError`/`hideError` (element renamed to `delete-wallet-error`, removed inconsistent `text-red-500`) - `addWallet.js`: replace `showFlash()` validation errors with dedicated `add-wallet-error` div - `importKey.js`: replace `showFlash()` validation errors with dedicated `import-key-error` div - `index.html`: add error divs for add-wallet and import-key views, rename flash elements to error elements `docker build .` passes. Closes #87
clawbot added 1 commit 2026-02-28 22:07:01 +01:00
fix: standardize error display to use showError/hideError helpers
check / check (push) Successful in 9s
Details
5e4cf34ef8 
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
clawbot closed this pull request 2026-02-28 22:07:10 +01:00
clawbot commented 2026-02-28 22:07:11 +01:00
Author
Collaborator
Copy Link
Copy Source

Closing as duplicate of #88.

Closing as duplicate of #88.

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#90