Skip to content

fix(ui): enable keyboard submission for dialogs #760#2005

Merged
rolznz merged 6 commits into
getAlby:masterfrom
keshav0479:fix/dialog-keyboard-submit
Jan 12, 2026
Merged

fix(ui): enable keyboard submission for dialogs #760#2005
rolznz merged 6 commits into
getAlby:masterfrom
keshav0479:fix/dialog-keyboard-submit

Conversation

@keshav0479
Copy link
Copy Markdown
Contributor

@keshav0479 keshav0479 commented Jan 6, 2026

Description

Fixes #760
Fixes #1976

This PR improves accessibility and UX by ensuring that dialogs with input fields can be submitted using the Enter key. Previously, users were required to manually click the "Confirm" button.

Changes

  • Wrapped input fields and action buttons in a <form> element across multiple dialog components.
  • Added onSubmit handlers to trigger the confirm action via keyboard.
  • Updated "Confirm" buttons to type="submit".

Files Updated

  • ResetRoutingDataDialogContent.tsx
  • RoutingFeeDialogContent.tsx
  • ExecuteCustomNodeCommandDialogContent.tsx
  • DebugTools.tsx (Fixed ProbeInvoice, ProbeKeysend, RefundSwap, GetLogs, and GetNetworkGraph dialogs)

@rolznz rolznz added this to the v1.21.3 milestone Jan 7, 2026
Comment thread frontend/src/components/ExecuteCustomNodeCommandDialogContent.tsx Outdated
@rolznz rolznz modified the milestones: v1.21.3, v1.22.0 Jan 7, 2026
@keshav0479 keshav0479 force-pushed the fix/dialog-keyboard-submit branch from 7a628a2 to 6a6aecb Compare January 8, 2026 20:16
Copy link
Copy Markdown
Member

@rolznz rolznz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

There was an issue with the routing fee dialog I fixed. I guess you didn't test this one (needs a public channel)

I made some other fixes too, and removed some unused code.

@rolznz rolznz merged commit 797faa7 into getAlby:master Jan 12, 2026
6 of 10 checks passed
@rolznz rolznz modified the milestones: v1.22.0, v1.21.4 Jan 12, 2026
@keshav0479
Copy link
Copy Markdown
Contributor Author

Thanks for the merge! And nice catch on the routing fee dialog, i couldn't verify that one locally, so i appreciate you fixing it.

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.

Bug: debug tools node command dialog has no scrollbar [Accessibility] Some dialogs are not easily submitted via keyboard

2 participants