Make WordPress Core

Opened 16 months ago

Last modified 13 months ago

#48536 accepted enhancement

Allow cmd/ctrl-enter to submit comment forms in wp-admin

Reported by: david.binda Owned by: davidbaumwald
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-patch needs-testing 2nd-opinion
Focuses: javascript, administration Cc:


This is a follow-up on #41545 which says:

I think it'd be a nice enhancement for comment forms in both the admin and front end to submit this way.

However, the r45790 introducing the feature does so only for frontend. Submitting comment by pressing ctrl/cmd + 'enter' does not seem to be working in wp-admin.

It would be cool if the new feature could be added to wp-admin as well.

Attachments (3)

48536.diff (636 bytes) - added by davidbaumwald 15 months ago.
Updated with keypress
48536.2.diff (637 bytes) - added by davidbaumwald 13 months ago.
Refreshed patch
48536.3.diff (848 bytes) - added by davidbaumwald 13 months ago.

Download all attachments as: .zip

Change History (12)

#1 @SergeyBiryukov
16 months ago

  • Focuses administration added

This ticket was mentioned in Slack in #core by peterwilsoncc. View the logs.

15 months ago

#3 @ehtis
15 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 5.4

Thanks for the report @davidbinda!

That indeed appears to be an issue - for now setting it for 5.4.

#4 @davidbaumwald
15 months ago

  • Owner set to davidbaumwald
  • Status changed from new to accepted

#5 @davidbaumwald
15 months ago

  • Keywords has-patch needs-testing added; needs-patch removed

15 months ago

Updated with keypress

13 months ago

Refreshed patch

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.

13 months ago

#7 @davidbaumwald
13 months ago

After some help from @garrett-eclipse, I'm updating the patch for WPCS and dropping the '10' keyCode.

#8 @davidbaumwald
13 months ago

  • Keywords 2nd-opinion added

Revisiting the 10 keyCode I had in the original patch, this was to accommodate Chrome's quirk for the keypress event. See https://bugs.chromium.org/p/chromium/issues/detail?id=79407. I'm thinking the final patch should include this, and we should also re-open #41545 or create a new ticket. Marking for 2nd-opinion.

#9 @davidbaumwald
13 months ago

  • Milestone changed from 5.4 to Future Release

This ticket still needs a final direction, patch, and testing. With 5.4 Beta 1 landing tomorrow, this is being moved to Future Release. If any maintainer or committer feels this can be included in 5.4 or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

Note: See TracTickets for help on using tickets.