Cmd+S needs to work in text view of post editor

[krogsgard]

Cmd/Control+S works in the visual editor, but not the text view of the WordPress post editor. It should work in either. If you do cmd+S in text view, it prompts to save a file. This is not preferred.

[valendesigns]

I can't save in Visual or Text mode with Command+S, both prompt a file save. What browser are you using? I tested in Chrome, Safari, and Firefox running OSX.

[voilamedia]

Where in core is this handled? i may be able to patch

[voilamedia]

Something like this should work

document.addEventListener("keydown", function(e) {
  if (e.keyCode == 83 && (navigator.platform.match("Mac") ? e.metaKey : e.ctrlKey)) {
    e.preventDefault();
    // Process save
  }
}, false);

i believe this should fix things.. but seeing the core uses keypress i cant seem to be able to find where the save is being made in the admin js files. Can someone point me to the file and line number where this is being done.

[iseulde]

I wonder if the user will then expect everything on the page to save. At the moment your cursor needs to be in the visual editor to use it and it will only save the main fields. Should it work for the whole page, or only when the cursor is in one of the editors?

[azaozz]

Thinking this should work the same as in the Visual editor, i.e. run autosave.

Can enhance it to include other content after we decide what we are going to do for #7756. If we go with:

$form.attr( 'target', 'some-hidden-iframe' ).submit().attr( 'target', '' );

we wouldn't need anything more.

[azaozz]

In 32680:

Editor: run autosave when pressing Ctrl or Command + S in the Text editor, same as the Visual editor.
Fixes #31655.

[johnbillion]

Replying to iseulde:

I wonder if the user will then expect everything on the page to save. At the moment your cursor needs to be in the visual editor to use it and it will only save the main fields. Should it work for the whole page, or only when the cursor is in one of the editors?

This is a concern. There's no reason for this only to trigger while focus is in the editor, and performing an autosave isn't what I'd expect from cmd+S (although #7756 would address this in the longer term).

TIL that cmd/ctrl+S was quietly introduced for autosave in the editor back in #24067.

cmd+S should perform a complete save, and should trigger regardless of where the focus is on the page.

Tagging accessibility because I know that custom keyboard shortcuts potentially have an accessibility impact.

[helen]

Replying to johnbillion:

cmd+S should perform a complete save, and should trigger regardless of where the focus is on the page.

You want cmd+s to trigger a full page reload? I still press cmd+s compulsively even though it's not 1998 anymore, surely I can't be the only one. Having it go full-page-reload would drive me to tears, and curious what it would do for a published post, then.

[Kelderic]

Replying to helen:

Replying to johnbillion:

cmd+S should perform a complete save, and should trigger regardless of where the focus is on the page.

You want cmd+s to trigger a full page reload? I still press cmd+s compulsively even though it's not 1998 anymore, surely I can't be the only one. Having it go full-page-reload would drive me to tears, and curious what it would do for a published post, then.

It should probably be dependent on #7756. Once we've changed to saving without a page reload, it won't be so intrusive to the user.

[obenland]

@azaozz, what do you want to do here? Revert and wait for #7756? Or close and create a new ticket for a complete save once #7756 is in?

[azaozz]

Yeah, thinking this is done for now. It works exactly the same in both Text and Visual editors: the editor area has to have focus, and we do an autosave.

Best is to revisit it (in a new ticket) after #7756.