Skip to content

Web: focus the form on IBAN, guard the amount, close the review's open points - #3

Merged
bmmmm merged 1 commit into
mainfrom
web-focus
Sep 21, 2026
Merged

bmmmm merged 1 commit into
mainfrom
web-focus

Conversation

@bmmmm

@bmmmm bmmmm commented Sep 21, 2026

Copy link
Copy Markdown
Owner

What

  • Form: name, IBAN, amount, remittance text up front; reference, BIC, purpose code, note to the payer under a folded "More fields" (a share link naming one opens it).
  • Amount: typed letters are dropped via beforeinput; pasted text still reaches the Go validator.
  • Review leftovers: link format version checked (v=1), flash() label race, live region narrowed, DE translations for switcher/buttons/summary, render.DefaultPNGScale shared by --png and the web download.
  • web-smoke.mjs greps web/* for storage APIs, address-bar writes and external resources.
  • README: intro, Install, Web and Development updated; CHANGELOG entry.

Verified

  • gofmt, go vet (native and js/wasm), go test ./..., scripts/build-web.sh, node scripts/web-smoke.mjs green.
  • Grep gate shown red with a localStorage call and an external <link>.
  • Chrome on web/dist: fold closed by default and opened by a bic= link, letter typed into amount refused while ,//paste pass, v=2 link refused with a message, labels restored in the current language after flash().

…n points

The form shows name, IBAN, amount and remittance text; reference, BIC,
purpose code and the note to the payer fold under "More fields" (IBAN-only
is the SEPA norm; a share link naming one of them opens the fold). Typed
letters never land in the amount field, pasted ones reach the validator.

Review leftovers: a share link from a newer format version is refused;
flash() restores button labels from the i18n table; the live region is
narrowed to status and error; the language switcher, download buttons and
summary translate; the PNG scale is one constant (render.DefaultPNGScale)
shared by --png and the web download.

web-smoke.mjs now also greps the page sources for storage APIs, address-bar
writes and external resources; both a storage call and an external
stylesheet were shown to turn it red. README documents the web version in
the intro, Install, Web and Development sections.
@bmmmm
bmmmm merged commit 5cb5275 into main Sep 21, 2026
4 checks passed
@bmmmm
bmmmm deleted the web-focus branch September 21, 2026 09:24

This branch was successfully deployed

1 active deployment
github-pages 5cb52758 Deployed Sep 21, 2026 by bmmmm via deploy #2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant