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

style(ui): fix home page responsiveness #37

Merged

Conversation

arjunan-k
Copy link
Contributor

Pull Request: Responsive Issue Fix

Description

This PR addresses a responsive issue mentioned in #28 where the UI breaks between the 1440px and 600px viewport width range. It includes changes to CSS Tailwind classes and some modifications to the TSX/HTML code to ensure a consistent and responsive user experience. Additionally, I have a video demonstration of the changes for a more detailed view.

Changes Made

  • Updated CSS Tailwind classes to improve responsiveness.
  • Modified TSX/HTML code to ensure proper rendering across different screen sizes.

UI Before

before.mp4

UI After

after.mp4

Related Issues

Closes #28 (if applicable)

Hi @SaraVieira and @jfranciscosousa 👋, could either of you please review this PR?
Feedback and suggestions are welcome, Thank you 😊!

@vercel
Copy link

vercel bot commented Sep 21, 2023

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

Name Status Preview Comments Updated (UTC)
react-url-modal ❌ Failed (Inspect) Sep 21, 2023 4:01pm

@jfranciscosousa
Copy link
Member

Hey @arjunan-k thanks for the PR. I'll give it a check later today. Let me just see what's happening with the Vercel preview first.

@jfranciscosousa jfranciscosousa self-assigned this Sep 22, 2023
@jfranciscosousa jfranciscosousa merged commit caca2af into remoteoss:main Sep 22, 2023
2 checks passed
@jfranciscosousa
Copy link
Member

Thanks for the contribution!

@arjunan-k
Copy link
Contributor Author

Thanks for the contribution!

@jfranciscosousa, You're welcome :)

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.

Responsive bug in some screen sizes
2 participants