Skip to content

fix(ui): chat section and chat message#196

Merged
ArnavAgrawal03 merged 1 commit into
morphik-org:mainfrom
abhix4:fix/chat-section
Jun 13, 2025
Merged

fix(ui): chat section and chat message#196
ArnavAgrawal03 merged 1 commit into
morphik-org:mainfrom
abhix4:fix/chat-section

Conversation

@abhix4

@abhix4 abhix4 commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

before:
Screenshot from 2025-06-11 16-48-45

after:
image

@jazzberry-ai

jazzberry-ai Bot commented Jun 11, 2025

Copy link
Copy Markdown

Bug Report

Chat message alignment and width issues
Medium

  1. Open the chat interface. 2. Send a long message as a user. 3. Receive a long message from the assistant. 4. Observe the alignment and width of the messages on different screen sizes.
    The changes in ChatMessages.tsx and ChatSection.tsx might introduce alignment and width issues in the chat message display.

Specifically:

  • The removal of ml-auto from the user's message container might cause the user's messages to be left-aligned instead of right-aligned.
  • The addition of pr-10 to the assistant's message container might cause the assistant's messages to have excessive padding on the right side.
  • The addition of max-w-4xl to the inner div in ChatMessages.tsx might cause overflow issues on smaller screens if the message content is too long.
  • The removal of px-4 sm:px-6 from the input area's container in ChatSection.tsx might cause the input area to touch the edges of the screen on smaller screens, reducing responsiveness.

Comments? Email us.

@ArnavAgrawal03

Copy link
Copy Markdown
Collaborator

has the font changed? we prefer the older font, looks good otherwise!

@abhix4

abhix4 commented Jun 13, 2025

Copy link
Copy Markdown
Contributor Author

@ArnavAgrawal03 no font changes . its my default ubuntu machine

@ArnavAgrawal03 ArnavAgrawal03 merged commit 33dae81 into morphik-org:main Jun 13, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants