Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#25771 closed defect (fixed)

Make the consensus health individual relay load trigger on enter

Reported by: teor Owned by: tom
Priority: Medium Milestone:
Component: Metrics/Consensus Health Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Usually, when I want to submit a form after typing in a field, I can just type enter. But this doesn't work on the consensus health individual relay load text box.

Child Tickets

Change History (4)

comment:2 Changed 2 years ago by teor

Thanks. Out of curiosity (it's been years since I did web programming), why not use "input type=submit"?
https://stackoverflow.com/questions/7210678/how-to-make-enter-the-submit-button-in-a-form#7210693

(The keycode seems to work on iOS, but the accessibility/discoverability is worse, because the "return" button doesn't turn blue and say "Go".)

comment:3 Changed 2 years ago by tom

It's more complicated. 'submit' only works if you're inside a <form> block, and if you're inside a <form> block but you don't want to actually _submit_ a form and cause a page reload you need to capture the form submission and suppress it.

I hadn't really considered any browser nicities that have been developed (especially on mobile) that make 'submit' more friendly than 'button' but a quick test confirmed all the annoying form stuff would still need to be handled.

comment:4 Changed 2 years ago by teor

No worries then. It's still functional.

Note: See TracTickets for help on using tickets.