Dun-sin / Whisper

A fun Application to have a random chat with people safely
https://whisper.favour.dev/
MIT License
390 stars 367 forks source link

[BUG] Scrollbar effect #536

Closed RANJEETJ06 closed 1 year ago

RANJEETJ06 commented 1 year ago

Description

in chat the scrollbar behaviour is weird looking

Screenshots

Screenshot (56)

Additional information

No response

🥦 Browser

Google Chrome

👀 Have you checked if this issue has been raised before?

🏢 Have you read the Contributing Guidelines?

Are you willing to work on this issue ?

None

github-actions[bot] commented 1 year ago

It's great having you contribute to this project by creating an issue

Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

Aryakoste commented 1 year ago

Hello @Dun-sin. I can work on this issue. But just want to ask if we should just remove scrollbar display by keeping scrolling on overflow or show scrollbar only when the content overflows the container.

Dun-sin commented 1 year ago

Hello @Dun-sin. I can work on this issue. But just want to ask if we should just remove scrollbar display by keeping scrolling on overflow or show scrollbar only when the content overflows the container.

Second one is a better choice. You've got it.

thanks for wanting to contribute. Follow the rules here, or your PR won't be accepted and will be closed. Good luck!

Aryakoste commented 1 year ago

image its showing this error for backend whenever i try to send message from anoynomous user

Dun-sin commented 1 year ago

image its showing this error for backend whenever i try to send message from anoynomous user

How latest is your local code

Aryakoste commented 1 year ago

Yesterday I was not getting the error. Now i forked the latest code from your repo and started getting this error.

Dun-sin commented 1 year ago

Yesterday I was not getting the error. Now i forked the latest code from your repo and started getting this error.

I understand, so you need to add VITE_SECRET_KEY AND SECRET_KEY for the backend and frontend respectively. The value can be anything

Aryakoste commented 1 year ago

Yes now working perfectly. I tested changes done by me and they are working. After the messages overflow the container then only the scrollbar appear. So I don't need to push anymore changes. Thank you.