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(ui): resolve chat container overlap with header #95

Merged

Conversation

AritraDey-Dev
Copy link
Contributor

Fixes Issue #92

Changes proposed

  • Fixed chat message container overlapping with header
  • Improved container layout and scroll behavior
  • Optimized spacing between UI elements
  • Added proper sticky positioning for chat components

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.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Screencast.from.2025-01-29.22-52-16.webm

Copy link
Owner

@sristy17 sristy17 left a comment

Choose a reason for hiding this comment

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

Great job, @AritraDey-Dev! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb here is the same PR for solving the chatbot overlap issue.

@pooranjoyb
Copy link
Collaborator

@pooranjoyb here is the same PR for solving the chatbot overlap issue.

Could you please check the overlapping issue around the input box, it's also overlapping :// @AritraDey-Dev

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb i have fixed the overlapping issue.Kindly look into this.
Screenshot from 2025-02-02 19-35-43

Copy link
Collaborator

@pooranjoyb pooranjoyb left a comment

Choose a reason for hiding this comment

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

Rest LGTM :)
Just remove the pnpm-lock.yaml file

@AritraDey-Dev
Copy link
Contributor Author

Rest LGTM :) Just remove the pnpm-lock.yaml file

I have removed the pnpm lock file now please check it once.

@pooranjoyb pooranjoyb merged commit b061f1d into sristy17:dwoc-2025 Feb 3, 2025
@sristy17 sristy17 added dwoc-2025 Issue under DWOC 2025 level 2 labels Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dwoc-2025 Issue under DWOC 2025 level 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants