#6521 closed defect (wontfix)
put save before cancel in admin forms
| Reported by: | Owned by: | Christopher Lenz | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | admin/web | Version: | devel |
| Severity: | normal | Keywords: | |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
The current behavior for many of the forms in WebAdmin is unexpected, at least in Firefox. When one hits the enter key, it cancels instead of submitting because the cancel button comes before the submit button. This patch flips them around.
Attachments (2)
Change History (8)
by , 17 years ago
| Attachment: | save_cancel.diff added |
|---|
follow-up: 3 comment:2 by , 17 years ago
Maybe there should also be a reset button, to revert the currently made changes..
comment:3 by , 17 years ago
Replying to anonymous:
Maybe there should also be a reset button, to revert the currently made changes..
I don't think it is worth cluttering the interface with a rarely used function that can be executed anyway with a simple page refresh.
comment:4 by , 16 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
There's a consistency issue here: all forms in Trac have a non-committing action as the default ("Preview" for wiki an ticket, "Cancel" for admin, etc.). This also avoids triggering "dangerous" actions inadvertently with ENTER.
I would say that this is a good thing (tm).
comment:5 by , 16 years ago
I'd say there's a big difference between preview and cancel, and cancel should never be the default action on a form, but I'll leave this at wontfix.
comment:6 by , 11 years ago
This issue is being discussed again in #11076, where this ticket has been mentioned.
save before cancel patch