Page MenuHomePhabricator

Make Cmd+Enter work to submit a form
ClosedPublic

Authored by sophiebits on Apr 4 2014, 1:59 AM.
Tags
None
Referenced Files
F18528038: D8699.diff
Sat, Sep 6, 8:05 PM
F18288140: D8699.diff
Sat, Aug 23, 8:09 AM
F18107171: D8699.id.diff
Aug 10 2025, 11:44 PM
F18045810: D8699.id20632.diff
Aug 3 2025, 9:56 AM
F17986144: D8699.id20633.diff
Aug 1 2025, 9:16 PM
F17956094: D8699.id.diff
Aug 1 2025, 5:19 AM
F17948544: D8699.diff
Jul 31 2025, 9:21 PM
F17857123: D8699.id20633.diff
Jul 27 2025, 6:07 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP077f57f06dee: Make Cmd+Enter work to submit a form
Summary

Fixes T3010.

keydown is what you're supposed to use for special keys like Enter; keypress didn't work for me for Cmd+Enter on Chrome.

Test Plan

Submitted a Differential field with Cmd+Enter on Chrome Mac. Also made sure Ctrl+Enter still works and Opt+Enter does not.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sophiebits added a task: Restricted Maniphest Task.
sophiebits retitled this revision from to Make Cmd+Enter work to submit a form.
sophiebits updated this object.
sophiebits edited the test plan for this revision. (Show Details)
sophiebits added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 4 2014, 2:00 AM
epriestley updated this revision to Diff 20633.

Closed by commit rP077f57f06dee (authored by @spicyj, committed by @epriestley).