Standardizes all password/validation error displays to use the `showError()`/`hideError()` helper pattern with `min-h-[1.25rem]` error divs.
## Changes
**HTML (`index.html`):**
- Added `add-wallet-error` div with `min-h-[1.25rem]` to add-wallet view
- Added `import-key-error` div with `min-h-[1.25rem]` to import-key view
- Renamed `export-privkey-flash` → `export-privkey-error` with `min-h-[1.25rem]`
- Renamed `delete-wallet-flash` → `delete-wallet-error` with `min-h-[1.25rem]`, removed `text-red-500`
**JS:**
- `addWallet.js`: validation errors use `showError("add-wallet-error", ...)` instead of `showFlash()`; success/status messages remain as `showFlash()`
- `importKey.js`: all validation errors use `showError("import-key-error", ...)` instead of `showFlash()`
- `addressDetail.js`: export-privkey errors use `showError()/hideError()` instead of direct DOM
- `deleteWallet.js`: errors use `showError()/hideError()` instead of direct DOM; "Wallet deleted" success remains as `showFlash()`
`docker build .` passes (tests + lint + build).
Closes #87
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
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Standardizes all password/validation error displays to use the
showError()/hideError()helper pattern withmin-h-[1.25rem]error divs.Changes
HTML (
index.html):add-wallet-errordiv withmin-h-[1.25rem]to add-wallet viewimport-key-errordiv withmin-h-[1.25rem]to import-key viewexport-privkey-flash→export-privkey-errorwithmin-h-[1.25rem]delete-wallet-flash→delete-wallet-errorwithmin-h-[1.25rem], removedtext-red-500JS:
addWallet.js: validation errors useshowError("add-wallet-error", ...)instead ofshowFlash(); success/status messages remain asshowFlash()importKey.js: all validation errors useshowError("import-key-error", ...)instead ofshowFlash()addressDetail.js: export-privkey errors useshowError()/hideError()instead of direct DOMdeleteWallet.js: errors useshowError()/hideError()instead of direct DOM; "Wallet deleted" success remains asshowFlash()docker build .passes (tests + lint + build).Closes #87
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 #87Closing — issue #87 is already closed (covered by merged PR #92).
Pull request closed