-
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: #1219 Too big top margin of message box separator #1385
Conversation
@AkashJana18 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 #1385 +/- ##
==========================================
+ Coverage 33.45% 34.03% +0.57%
==========================================
Files 64 69 +5
Lines 3273 3376 +103
==========================================
+ Hits 1095 1149 +54
- Misses 2178 2227 +49 ☔ View full report in Codecov by Sentry. |
Hey @ManishMadan2882 @tenokami can you please take a look? |
@AkashJana18 Great change! Thank you |
Hey @tenokami can you merge it as well, as today all the fest are ending. I would be glad if it merged before the fest ends. |
Hi @AkashJana18 |
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.
@AkashJana18
The scrollbar is removed from the input box, please check it once with larger inputs.
Thanks!
I have made the requested change @ManishMadan2882 can you please review it? |
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 @AkashJana18 !
Fixes: 🐛 Bug Report: Too big top margin of message box separator #1219
Before :
After:
Better UI/UX
Feel free to ask if anymore changes required