autoinfra-ai / lingtual-embedded-chat

easy embedded chat widget
https://ligtual.com
3 stars 3 forks source link

[Bounty- $25] mobile responsive #12

Open bhaktatejas922 opened 4 months ago

bhaktatejas922 commented 4 months ago

We need to make the innkeeper/ branch mobile responsive

this can be test using

npm run start -> using a chrome extension to simulate a phone

bhaktatejas922 commented 4 months ago

chat must be full screen on mobile. X to exit chat must also be visible

magiccoder-beep commented 4 months ago

I can do it.

magiccoder-beep commented 4 months ago

could you send me your design or file for responsive?

John-Weak commented 4 months ago

@bhaktatejas922 does this look good?

image
magiccoder-beep commented 4 months ago

so you have to implement your UI to responsive for mobile devices?

magiccoder-beep commented 4 months ago

Ok, I will do that

magiccoder-beep commented 4 months ago

send me invitation to github profile, redcrystalll

John-Weak commented 4 months ago

@blackberry2024 i have done the code changes already 😄 , only confirming the output with @bhaktatejas922

https://github.com/autoinfra-ai/lingtual-embedded-chat/pull/13

magiccoder-beep commented 4 months ago

Oh. so I have no things to do for your project?

bhaktatejas922 commented 4 months ago

Thanks for the update @John-Weak

Will test soon then send the bounty.

utin-francis-peter commented 4 months ago

Hello, @bhaktatejas922

I've implemented the requirements and made a PR. Kindly have a look.

PR link: https://github.com/autoinfra-ai/lingtual-embedded-chat/pull/14

bhaktatejas922 commented 4 months ago

Issue is still open as neither PR works correctly on mobile + desktop:

Addition context: - this is how we use it

<ChatWidget
        window_title={`${websiteInformation.hotelName || ''} Front Desk`}
        flow_id={websiteInformation.dashUserId || ''}
        host_url={process.env.NEXT_PUBLIC_HOTELAI_ROOT_DOMAIN}
        api_key={sessionId} // Use sessionId in the apiKey slot
        //@ts-ignore
        style={{
          position: 'fixed',
          bottom: '50px',
          right: '50px',
          zIndex: 999,
        }}
        chatInputField="input"
        chat_inputs='{"input":""}'
        suggested_questions='[
    {
      "questionId": 1,
      "text": "What is the check-in and check-out time?"
    },
    {
      "questionId": 2, 
      "text": "What is nearby?"
    },
    {
      "questionId": 3,
      "text": "What is the cancellation policy?"
    }
  ]'
      ></ChatWidget>
utin-francis-peter commented 4 months ago

@bhaktatejas922 please could you leave a comment in my PR?

Unsure where to look at the moment as new styles only applied to screens <= 768px, thereby maintaining the desktop styles as they initially were.

On mobile, the shared output was achieved.

Please share some steps on using the widget in a real application for better testing.

Thank you.

John-Weak commented 4 months ago

@bhaktatejas922 updated the PR

image image
jay7134 commented 4 months ago

I can do it.