Skip to content

Comments

fix(ui): resolve chat container overlap with header#95

Merged
pooranjoyb merged 4 commits intosristy17:dwoc-2025from
AritraDey-Dev:fix/chat-container-overlap
Feb 3, 2025
Merged

fix(ui): resolve chat container overlap with header#95
pooranjoyb merged 4 commits intosristy17:dwoc-2025from
AritraDey-Dev:fix/chat-container-overlap

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
@AritraDey-Dev AritraDey-Dev deleted the fix/chat-container-overlap branch April 17, 2025 20:03
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.

3 participants