unable to commit after cmd+enter

Description

Partial crash. Unable to commit after hitting cmd+enter when having altered a single file.

Version

• GitHub Desktop: 1.5.0

• Operating system: macOS 10.14.1 (18B75)

Steps to Reproduce

1. edit one file in repo.
2. hit cmd+enter

Expected Behavior

File gets committed with commit message “Updated foo.bar”.

This is what happens when you manually click “Commit to master” instead of using the cmd+enter keyboard shortcut. This is also what happens when you focus the Summary field and then hit cmd+enter.

Actual Behavior

“Commit to master” button is disabled, file is not committed. Client has to be restarted to be able to commit.

Additional Information

none

Logs

2018-12-03.desktop.production.log

[billygriffin]

Hi @fschroiff, we discussed this quite a bit internally and unfortunately the technical work to achieve something that captures the commit keyboard shortcut outside of the commit area is nontrivial, and we’re not sure this would provide benefit for most users since we haven’t heard this before. I totally understand how this could make things more convenient for you, but it’s not something we intend to support unless we hear evidence that it’s causing more widespread pain and/or confusion (or frustration from lack of convenience).

I really appreciate you describing your expectation of how this would work, and hope you’ll continue to weigh in on things you’d like to see improved or that are causing frustration for you in the future. Thanks again! 😄

One note for future reference: the toggle of the checkboxes is just caused by hitting the enter key alone.

[fschroiff]

@tierninho there might not have been a crash, I just interpreted the UI-state as being unresponsive. I had never even noticed the checkboxes ☑️ 👀. I learned now that focus on a file plus cmd+enter deselects the file (= un-checks the checkbox). And focus on the summary field plus cmd+enter triggers a commit. This is the source of my confusion.

So I’ll turn this from a bug-report into a feature request: When there’s an autogenerated summary (which is the case when only one file has been edited) cmd+enter should trigger a commit even when the files pane is focussed.