Fix: Adjust padding in chat view for better UI#2965
Conversation
|
|
CC @sachasayan |
|
@zhangtony239 I think padding is useful to give the content some room to breathe on larger widths. Let's try some variation of |
Indeed, it's time to add a condition to this CSS. |
2025-04-29.235736.mp4@sachasayan How about this? |
|
@zhangtony239 Love it. Let's ship this, but one minor nitpick: Instead of: Can you use: Keeping responsible styles consistently bottom up means conflicts / bugs later if they occur and we get to build defaults to the narrow sizes. 😇 Sorry to nitpick, and this is a great catch! Thanks for the contribution! 👍 |
I just noticed that this makes the p-10 take priority over the pt-5 for the top padding - is that intentional? |
|
Maybe we should just merge this as is |
|
@mrubens Let me check it out real quick. |
|
@mrubens It's fine as-is. Very little difference. I'll do another pass on this soon. I think you can just merge. :) |
Context
Adjust minimum padding for history card to accommodate small width layouts.
Screenshots
How to Test
Get in Touch
Discord: zhangtony239
Important
Adjust padding in
ChatView.tsxfor better UI on small width layouts.ChatView.tsxfromp-10top-3.5for better accommodation of small width layouts.This description was created by
for 44154d50e5f0687af12a6d3734c30eb8f4e362ba. You can customize this summary. It will automatically update as commits are pushed.