Skip to content

Fix Enter key handling for confirmation dialogs#129

Open
OnlyYu1996 wants to merge 1 commit into
x42protocol:masterfrom
OnlyYu1996:fix-enter-key-dialog-confirmation
Open

Fix Enter key handling for confirmation dialogs#129
OnlyYu1996 wants to merge 1 commit into
x42protocol:masterfrom
OnlyYu1996:fix-enter-key-dialog-confirmation

Conversation

@OnlyYu1996
Copy link
Copy Markdown

Fixes #31.

What changed

  • Use form submit handlers for wallet creation and secret-word confirmation so Enter triggers the same action as clicking the primary button.
  • Let the Secret Words dialog continue on Enter and the wallet-created OK state close on Enter.
  • Add Enter handling for the shared generic OK modal.
  • Focus the Receive dialog OK button by default so Enter closes it without intercepting other Receive actions.

Verification

  • yarn install --ignore-scripts --frozen-lockfile

px ng lint

px tsc -p tsconfig.app.json --noEmit

pm run postinstall:electron

  • $env:NODE_OPTIONS='--openssl-legacy-provider'; npx ng build --configuration dev

Note: the build needs NODE_OPTIONS=--openssl-legacy-provider on Node 20 because this Angular 10 / Webpack 4 project uses legacy OpenSSL hashing. The build completed with existing CommonJS optimization warnings.

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.

Minor Bug - 'Enter' key doesn't work in some boxes

1 participant