feat(surveys): submit open text questions with Cmd/Ctrl+Enter

.changeset/surveys-open-text-cmd-enter-submit.md

@@ -0,0 +1,5 @@
+---
+'posthog-js': patch
+---
+
+Surveys: submit open text questions with Cmd/Ctrl+Enter. The textarea still inserts a newline on plain Enter (native behaviour), matching the convention used by Slack, GitHub, Discord, and ChatGPT for multi-line inputs. Single-line "Other:" inputs continue to submit on plain Enter as before.

packages/browser/src/__tests__/extensions/surveys/question-types.test.tsx

@@ -73,6 +73,21 @@ describe('MultipleChoiceQuestion', () => {
             expect(baseProps.onSubmit).toHaveBeenCalledWith('Purple')
         })
 
+        it('submits open-ended choice on Enter key', () => {
+            const onSubmit = jest.fn()
+            const { getByText, container } = render(
+                <MultipleChoiceQuestion {...baseProps} onSubmit={onSubmit} question={singleChoiceQuestion} />
+            )
+
+            fireEvent.click(getByText('Other:'))
+
+            const openInput = container.querySelector('#surveyQuestion1Choice3Open') as HTMLInputElement
+            fireEvent.input(openInput, { target: { value: 'Purple' } })
+            fireEvent.keyDown(openInput, { key: 'Enter' })
+
+            expect(onSubmit).toHaveBeenCalledWith('Purple')
+        })
+
         it('focuses on open-ended input when selecting the option', () => {
             const { container, getByText } = render(
                 <MultipleChoiceQuestion {...baseProps} question={singleChoiceQuestion} />
@@ -287,7 +302,53 @@ describe('OpenTextQuestion', () => {
         expect(parentKeyDownHandler).not.toHaveBeenCalled()
     })
 
-    // Add other tests for OpenTextQuestion if needed...
+    it('does not submit on plain Enter (textarea inserts newline)', () => {
+        const onSubmit = jest.fn()
+        const { container } = render(
+            <OpenTextQuestion {...baseProps} onSubmit={onSubmit} question={{ ...openTextQuestion, optional: true }} />
+        )
+
+        const textarea = container.querySelector('textarea')
+        if (!textarea) throw new Error('Textarea not found')
+
+        fireEvent.input(textarea, { target: { value: 'Hello world' } })
+        fireEvent.keyDown(textarea, { key: 'Enter' })
+
+        expect(onSubmit).not.toHaveBeenCalled()
+    })
+
+    it.each([
+        ['metaKey', { metaKey: true }],
+        ['ctrlKey', { ctrlKey: true }],
+    ])('submits on Enter+%s when input is valid', (_label, modifier) => {
+        const onSubmit = jest.fn()
+        const { container } = render(
+            <OpenTextQuestion {...baseProps} onSubmit={onSubmit} question={{ ...openTextQuestion, optional: true }} />
+        )
+
+        const textarea = container.querySelector('textarea')
+        if (!textarea) throw new Error('Textarea not found')
+
+        fireEvent.input(textarea, { target: { value: 'Hello world' } })
+        fireEvent.keyDown(textarea, { key: 'Enter', ...modifier })
+
+        expect(onSubmit).toHaveBeenCalledWith('Hello world')
+        expect(onSubmit).toHaveBeenCalledTimes(1)
+    })
+
+    it('does not submit on Cmd/Ctrl+Enter when validation fails', () => {
+        const onSubmit = jest.fn()
+        const { container } = render(
+            <OpenTextQuestion {...baseProps} onSubmit={onSubmit} question={openTextQuestion} />
+        )
+
+        const textarea = container.querySelector('textarea')
+        if (!textarea) throw new Error('Textarea not found')
+
+        // Required question, empty input → invalid.
+        fireEvent.keyDown(textarea, { key: 'Enter', metaKey: true })
+        expect(onSubmit).not.toHaveBeenCalled()
+    })
 })
 
 describe('RatingQuestion', () => {

packages/browser/src/extensions/surveys/components/QuestionTypes.tsx

@@ -149,6 +149,10 @@ export function OpenTextQuestion({
                     }}
                     onKeyDown={(e) => {
                         e.stopPropagation()
+                        if (e.key === 'Enter' && (e.metaKey || e.ctrlKey) && !validationError) {
+                            e.preventDefault()
+                            isPreviewMode ? handlePreviewSubmit() : handleSubmit()
+                        }
                     }}
                     value={text}
                 />