-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Enhance layout in Conversation components #1383
Conversation
…nents; add user icon to ConversationBubble
@SamsShow is attempting to deploy a commit to the Arc53 Team on Vercel. A member of the Team first needs to authorize it. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1383 +/- ##
==========================================
+ Coverage 33.63% 33.83% +0.19%
==========================================
Files 65 67 +2
Lines 3297 3334 +37
==========================================
+ Hits 1109 1128 +19
- Misses 2188 2206 +18 ☔ View full report in Codecov by Sentry. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @SamsShow
Could you please check the avatar for mobile responsiveness.
Think it would be better if placed like the dino icon next to Answer.
@ManishMadan2882 |
@ManishMadan2882 made the changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SamsShow
Sorry for not being clear before, only wanted the icon to be positioned like that on mobile screens, also the Prompt
text is not required.
Just need a minor change to make the Avatar align as it was before
fixes:
2)
4)
🐛 Bug Report: Basic UI changes #1310