Skip to content

fix: remove left padding from chat messages - #808

Merged
bdjulbic merged 2 commits into
mainfrom
fix/remove-padding-from-agent-chat
Jul 14, 2026
Merged

fix: remove left padding from chat messages#808
bdjulbic merged 2 commits into
mainfrom
fix/remove-padding-from-agent-chat

Conversation

@bdjulbic

@bdjulbic bdjulbic commented Jul 13, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Style
    • Updated spacing/alignment for collapsed activity summary toggle to remove the left margin.
    • Refined message container horizontal padding, with error messages now using distinct horizontal spacing compared to non-error types.
    • Adjusted chart reasoning text styling to remove the left margin for consistent alignment.
    • Preserved existing content, layout behavior, and functionality.
Screenshot 2026-07-13 at 21 48 33

@coderabbitai

coderabbitai Bot commented Jul 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Three component styling updates adjust left and horizontal spacing utilities. Error messages now receive explicit horizontal padding, while rendering logic remains unchanged.

Changes

UI spacing adjustments

Layer / File(s) Summary
Component spacing class updates
packages/ai-core/src/components/ActivityBox.tsx, packages/ai-core/src/components/MessageContainer.tsx, packages/vega/src/VegaChartToolResult.tsx		 Removed ml-4 from activity and reasoning elements, removed base px-5 from the message container, and added px-4 for error messages. Import formatting was also updated without changing symbols.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • sqlrooms/sqlrooms#576: Introduces the ActivityBox and Vega chart result UI elements affected by these spacing changes.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: reducing left-side spacing in chat/message UI components.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/remove-padding-from-agent-chat

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

dmitriy-kostianetskiy
dmitriy-kostianetskiy reviewed Jul 14, 2026
View reviewed changes
Comment thread packages/ai-core/src/components/MessageContainer.tsx
Comment on lines 34 to 35
// borderColor,
// isSuccess ? borderColor : 'border-red-500',

@dmitriy-kostianetskiy dmitriy-kostianetskiy Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: can be removed I guess?

@bdjulbic
bdjulbic merged commit 5c66a3c into main Jul 14, 2026
6 checks passed
@bdjulbic
bdjulbic deleted the fix/remove-padding-from-agent-chat branch July 14, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmitriy-kostianetskiy dmitriy-kostianetskiy dmitriy-kostianetskiy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bdjulbic @dmitriy-kostianetskiy