Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Review Summary by QodoReduce email capture widget size and improve visibility
WalkthroughsDescription• Reduce email capture widget padding and spacing • Remove "Skip" button from email capture form • Show email capture without requiring visitor message • Adjust footer and input container spacing Diagramflowchart LR
A["Email Capture Widget"] -->|"Reduce padding"| B["Smaller visual footprint"]
A -->|"Remove Skip button"| C["Encourage email submission"]
A -->|"Remove message requirement"| D["Show earlier in conversation"]
B --> E["Less obtrusive UI"]
C --> E
D --> E
File Changes1. apps/widget/src/styles.css
|
Code Review by Qodo
1.
|
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Comment on lines
+130
to
+132
| {/* <button onClick={onEmailDismiss} className="opencom-email-skip" type="button"> | ||
| Skip | ||
| </button> | ||
| </button> */} |
There was a problem hiding this comment.
2. Email prompt undismissable 🐞 Bug ✓ Correctness
The UI no longer renders any control that calls handleEmailDismiss, so users cannot dismiss email capture without submitting an email and the opencom_email_dismissed mechanism becomes unreachable. This can leave the email capture permanently visible for users who don’t want to provide an email.
Agent Prompt
### Issue description
Email capture can no longer be dismissed because the Footer UI no longer calls `onEmailDismiss`, even though ConversationView still implements dismissal behavior.
### Issue Context
`handleEmailDismiss` persists a dismissal flag in sessionStorage (`opencom_email_dismissed`). With the Skip button removed, that path is unreachable from the UI.
### Fix Focus Areas
- apps/widget/src/components/conversationView/Footer.tsx[96-134]
- apps/widget/src/components/ConversationView.tsx[252-283]
- apps/widget/src/components/ConversationView.tsx[32-41]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Contributor
There was a problem hiding this comment.
Pull request overview
Adjusts the widget conversation UI to make the email capture prompt feel less visually prominent by tightening spacing and simplifying the footer layout.
Changes:
- Tweaked padding in the messages area, input container, and email capture container.
- Removed footer spacing (
gap) between stacked footer elements. - Commented out the “Skip” action for email capture and relaxed the email-capture gating condition.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| apps/widget/src/styles.css | Reduces padding/spacing around messages, footer, and email capture UI. |
| apps/widget/src/components/conversationView/Footer.tsx | Removes (comments out) the email “Skip” button in the footer UI. |
| apps/widget/src/components/ConversationView.tsx | Changes email capture eligibility behavior by removing the “visitor has sent a message” gate. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
packages/react-native-sdk/src/components/messenger/ConversationDetailView.tsx
Show resolved
Hide resolved
djanogly
commented
Mar 12, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Djanogly <45178753+djanogly@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Make email collection less obtrusive, but persistent.