Ticket #181 (closed enhancement: fixed)

Opened 6 months ago

Last modified 5 months ago

In all forms, pressing the Enter key should trigger form validation

Reported by: tristan Owned by: fabian@…
Priority: minor Milestone: V2 Final
Component: Calenco UI Version: 2.0
Keywords: javascript Cc:

Description (last modified by fabian@…) (diff)

All forms (publications mgmt, user mgmt, workspace creation, etc.) should have the Enter key bound to form validation. This way, the user can press the Enter key to check form validtiy (useful for complex forms, like the pub creation one, for example) without the need to click on the corresponding form submit button.

Change History

comment:1 Changed 6 months ago by fabian@…

Well, not really, can you please provide more details? Which form? (or action, like adding a publication, editing a publication, adding a user, etc.)

And what does "Active Return key" mean? That the Enter key should be blocked and it's not? Something else?

Thanks in advance.

comment:2 Changed 6 months ago by tristan

This is a global "problem". In each forms (like publication, adding an user, etc.), we must use the mouse to validate the form. An user can't validate the form hitting the Return key.

The soluce is to detect this hit using javascript (there is a simple way in jquery, the equivalent should be in dojo) and then activate the form validation.

comment:3 Changed 6 months ago by fabian@…

  • Description modified (diff)
  • Summary changed from Active Return key in forms validation to In all forms, pressing the Enter key should trigger form validation

Ah... I see.

Yes, there's a dojo way to detect the Enter keypress on form input fields.

Even more, I've commited some fixes to the UI yesterday that uses this technique to actually block the Enter key in forms (See tkt #155, it was submitting the form on Firefox).

Maybe we can change that behaviour to fire the form validation (many forms lack those checks right now, no need to open another ticket, we'll use this one) instead.

I've rephrased the ticket. Please let me know if it's OK now.

comment:4 Changed 6 months ago by tristan

OK ;-)

comment:5 Changed 5 months ago by fabian@…

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in svn rev 744

Note: See TracTickets for help on using tickets.