8000 Fix UI for mobile devices by ManishMadan2882 · Pull Request #1132 · arc53/DocsGPT · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix UI for mobile devices #1132

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 20, 2024
Merged

Fix UI for mobile devices #1132

merged 5 commits into from
Sep 20, 2024

Conversation

ManishMadan2882
Copy link
Collaborator
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    1. UI adjustments on mobile devices
    2. Restructured the components in Conversation
  • Why was this change needed? (You can also link to an open issue here)
    Responsiveness

@vercel Vercel
Copy link
vercel bot commented Sep 17, 2024

@ManishMadan2882 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@vercel Vercel
Copy link
vercel bot commented Sep 17, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 19, 2024 10:27pm

@ManishMadan2882 ManishMadan2882 marked this pull request as ready for review September 18, 2024 08:18
@siiddhantt
Copy link
Collaborator

hi @ManishMadan2882 the scroll-to-bottom button doesn't work quite well for longer responses, if you could check it out. thanks!

Screen.mov

@ManishMadan2882
Copy link
Collaborator Author

Hi @siiddhantt
Could you please make sure you're reviewing the correct version? The scroll-to-bottom button is currently not visible in the version I'm working on, which is the main issue that can be addressed in this PR.
Thanks!

dartpain
dartpain approved these changes Sep 20, 2024
View reviewed changes
Copy link
Contributor
@dartpain dartpain left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@dartpain dartpain merged commit a9a4d14 into arc53:main Sep 20, 2024
7 checks passed
@ManishMadan2882 ManishMadan2882 mentioned this pull request Oct 3, 2024
Closed
@ManishMadan2882 ManishMadan2882 mentioned this pull request May 26, 2025
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dartpain dartpain dartpain approved these changes

Assignees
No one assigned
Labels
frontend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ManishMadan2882 @siiddhantt @dartpain
0