Skip to content
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: horizontal scroll #1156

Merged
merged 7 commits into from
Mar 14, 2024
Merged

Conversation

pritipsingh
Copy link
Contributor

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Copy link

github-actions bot commented Mar 6, 2024

  • In the ChatSupportTest component, there is a typo in the align-items property in the SenderMessage styled component. It should be flex-start instead of flex-strt.

  • There are commented out sections of code in the Chats component related to handling different message types such as 'Image' and 'File'. These sections should be reviewed and either uncommented and corrected or removed completely based on their relevance.

  • There is a missing closing curly brace } at the end of the Chats component before the //styles comment.

Other than these issues, the code looks fine.

All looks good.

Copy link
Collaborator

@mishramonalisha76 mishramonalisha76 left a comment

Choose a reason for hiding this comment

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

image @pritipsingh i still see horizontal scroll

Copy link
Collaborator

@mishramonalisha76 mishramonalisha76 left a comment

Choose a reason for hiding this comment

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

image the chatprofile coming on top has to be fixed from dapp side i guess, rest is fine

Copy link
Collaborator

@mishramonalisha76 mishramonalisha76 left a comment

Choose a reason for hiding this comment

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

image Text also excceeds

@mishramonalisha76 mishramonalisha76 merged commit 8fb3301 into alpha Mar 14, 2024
1 check failed
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