continue icon indicating copy to clipboard operation
continue copied to clipboard

feat: restore feedback modal for user interest collection

Open continue[bot] opened this issue 2 months ago • 3 comments

Summary

Restores the feedback modal that was removed in PR #6533 to help identify Continue knowledgeable/friendly users for CAI user interviews.

Changes

  • Restored FeedbackDialog component with modern, improved styling using Tailwind CSS
  • 🎨 Enhanced UX with:
    • Clean, card-based layout
    • "Not now" dismiss button for less intrusive experience
    • Auto-close modal after successful submission
    • Better visual hierarchy and spacing
  • 📊 Restored tracking logic:
    • LocalStorage counter that tracks user interactions
    • Triggers modal after 300 interactions
    • Captures name and email via PostHog user_interest_form event
  • 🔧 State management restored in Redux uiSlice

Testing

The modal will appear after 300 chat interactions. To test manually:

  1. Set mainTextEntryCounter in localStorage to 299
  2. Send one more chat message
  3. Verify the modal appears with improved styling

Related

  • Closes: CON-4856
  • Related PR: #6533 (removal)
  • Slack thread: https://continuedev.slack.com/archives/C060AV775UZ/p1762571766597529

This agent session was co-authored by dallin and Continue.


Summary by cubic

Restores the feedback modal to collect user interest for CAI interviews. It triggers after 300 chat interactions and submits name/email via PostHog, meeting the requirements in CON-4856.

  • New Features

    • Reintroduced FeedbackDialog with modern Tailwind styling, “Not now” dismiss, and auto-close after submit.
    • Tracks interactions via localStorage and triggers the modal at 300; captures details with user_interest_form.
    • Adds mainTextEntryCounter to localStorage types and uses Redux dialog state (setShowDialog, setDialogMessage).
  • Refactors

    • Removed unused dialogEntryOn state from uiSlice.

Written for commit da7baa55ed2185d4d9a6030e82896e1dc1880b9e. Summary will update automatically on new commits.

continue[bot] avatar Nov 08 '25 03:11 continue[bot]

Test Failure Analysis

The macOS test failures in TUIChat.editMessage.test.tsx are unrelated to this PR's changes:

All relevant checks passed:

  • lint ✅
  • gui-checks ✅
  • core-checks ✅
  • prettier-check ✅

Failing tests are in CLI (extensions/cli):

  • src/ui/__tests__/TUIChat.editMessage.test.tsx
  • Only failing on macOS (18, 20, 22) - macOS 24 passed
  • All Ubuntu and Windows tests passed

📁 This PR only modifies GUI files:

  • gui/src/components/dialogs/FeedbackDialog.tsx (new file)
  • gui/src/pages/gui/Chat.tsx
  • gui/src/redux/slices/uiSlice.ts
  • gui/src/util/localStorage.ts

The test failures appear to be a flaky test issue specific to certain macOS versions, not caused by this PR's changes.

continue[bot] avatar Nov 08 '25 03:11 continue[bot]

image

Also noting I checked posthog and it works

RomneyDa avatar Nov 11 '25 02:11 RomneyDa

red outlines around fields are only because required fields and I clicked submit

RomneyDa avatar Nov 11 '25 02:11 RomneyDa

:tada: This PR is included in version 1.5.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 19 '25 04:11 sestinj

:tada: This PR is included in version 1.31.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 19 '25 22:11 sestinj

:tada: This PR is included in version 1.35.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 20 '25 21:11 sestinj

:tada: This PR is included in version 1.6.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sestinj avatar Nov 20 '25 21:11 sestinj