Reviewing PRs: Ctrl+Enter submits comment instead of starting a review

[drsybren]

Description

When writing an inline comment on a pull request, the "Start review" button is highlighted as the primary button for the text area:

However, when pressing Ctrl+Enter on the keyboard, it is interpreted as pressing the "Add single comment" button.

The biggest issue here is that the "Add single comment" button immediately submits the comment. There is no way to undo this, whereas accidentally starting a review does not immediately send something to the PR author, and thus the review can be cancelled at a later time without bothering anyone, in case it was started by accident.

Gitea Version

1.19.0+dev-649-g2093ca517

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

Self-built from Git

Database

None

[wxiaoguang]

This behavior comes from

• Make Ctrl+Enter (quick submit) work for issue comment and wiki editor #19729

Why the Ctrl+Enter doesn't trigger the default primary button: because in the review form, users might not want to approve the PR, they might just want to leave a comment.

I agree to improve the UX, while some details could be fine-tuned and considered together.

A quick fix could be: change some forms' default actions, some forms use "start review" while some forms use "comment"

[wxiaoguang]

Quickly go through the code again, making the "Start Review" button as the default action seems not an easy task (not just minutes, but maybe hours?), maybe that's the reason why I didn't touch them last time when I added the Ctrl+Enter support .... I can still see my comment for name="is_review" value="true" in code 😂

---

Update: I tried to proposed a fix (more complicated than I thought ...), did some quick tests, it seems working as expected.

[wxiaoguang]

Please test this fix: Make Ctrl+Enter submit a pending comment (starting review) instead of submitting a single comment #23245