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

Responsiveness chatbot UI for mobile #442 #448

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

soubhik-111
Copy link
Contributor

Summary

The Issue is already listed in #442 with proper screenshots and captions.

Fixes # (issue)

Chatbot is breaking in the case of the mobile view.

Type of change

Please select the relevant option.

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe)

Checklist

Please tick the relevant options.

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Screenshots

Provide screenshots to help explain your changes.

Before change:

It is breaking i.e. half the chatbot is not showing on mobile.
249515264-029edddd-373b-4f17-8e76-d02585e5e985

After change:

It is completely visible on all mobiles.
image

Additional context

Please review and give your feedback.
Close #442
GSSOC'23

@kartikayasija
Copy link
Collaborator

Make it responsive for landscape view.
image

@soubhik-111
Copy link
Contributor Author

@kartikayasija
Thanks for your feedback have added it now.
Kindly Review it.

Screenshot:

device: Samsung s20 ultra

Portait:
image

Landscape:
image

device: iphone13

Portrait:
image

Landscape:
image

Copy link
Collaborator

@kartikayasija kartikayasija left a comment

Choose a reason for hiding this comment

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

Recording.2023-07-01.222041.mp4

Reverse the direction of animation.

@soubhik-111
Copy link
Contributor Author

@kartikayasija
Kindly review and merge it or give your feedback.

@soubhik-111
Copy link
Contributor Author

@kartikayasija Kindly merge it under GSSOC'23

@soubhik-111
Copy link
Contributor Author

@AbhiPatel10
Can you merge it

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.

AI Bot responsiveness
2 participants