Skip to content

Conversation

@continue
Copy link
Contributor

@continue continue bot commented Nov 8, 2025

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


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 da7baa5. Summary will update automatically on new commits.

- Restore FeedbackDialog component with improved modern styling
- Add Tailwind CSS classes for better UI/UX
- Include 'Not now' button for less intrusive experience
- Auto-close modal after submission
- Restore localStorage counter tracking in Chat.tsx
- Restore UI state management in uiSlice.ts
- Track user interest via PostHog 'user_interest_form' event

Closes CON-4856

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
Co-authored-by: dallin <[email protected]>
@continue
Copy link
Contributor Author

continue bot commented Nov 8, 2025

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 and others added 2 commits November 11, 2025 01:42
The dialogEntryOn state was not being read anywhere in the codebase,
only set once during the feedback dialog trigger. This commit removes:

- dialogEntryOn from UIState type definition
- dialogEntryOn from DEFAULT_UI_SLICE initial state
- setDialogEntryOn reducer action
- setDialogEntryOn export from uiSlice
- setDialogEntryOn import and dispatch call from Chat.tsx

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
Co-authored-by: Dallin Romney <[email protected]>
@RomneyDa RomneyDa marked this pull request as ready for review November 11, 2025 01:56
@RomneyDa RomneyDa requested a review from a team as a code owner November 11, 2025 01:56
@RomneyDa RomneyDa requested review from sestinj and removed request for a team November 11, 2025 01:56
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 11, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

@RomneyDa
Copy link
Collaborator

image

Also noting I checked posthog and it works

@RomneyDa
Copy link
Collaborator

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

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 17 files

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 19, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Nov 19, 2025
@RomneyDa RomneyDa merged commit 9f2350c into main Nov 19, 2025
56 of 58 checks passed
@RomneyDa RomneyDa deleted the continue/con-4856 branch November 19, 2025 01:50
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 19, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 19, 2025
@sestinj
Copy link
Contributor

sestinj commented Nov 19, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants