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

Fixed the height of the boxes present on the homepage of the website #471

Merged

Conversation

himughanakash
Copy link
Contributor

@himughanakash himughanakash commented Oct 3, 2023

This pr fixed the height of the boxes on the homepage.
fixes #459

Summary by CodeRabbit

Style: Adjusted CSS styles for better mobile responsiveness

  • Reduced the min-height property of the .listContainer class from 17rem to 8.8rem to optimize space utilization on smaller screens.
  • Introduced a max-height property with a value of 295px to the .listContainer class, ensuring consistent display across various screen sizes.

@vercel
Copy link

vercel bot commented Oct 3, 2023

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

Name Status Preview Comments Updated (UTC)
free-web3-resources ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 3, 2023 6:25pm
free-web3-resources-1l9i ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 3, 2023 6:25pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 3, 2023

Walkthrough

The recent changes primarily focus on enhancing the responsiveness of the homepage features. The modifications in CSS styles adjust the height of the .listContainer class, ensuring optimal display on screens with a maximum width of 768 pixels or less.

Changes

File Summary
.../HomepageFeatures/styles.module.css The min-height property of the .listContainer class has been reduced from 17rem to 8.8rem. A new max-height property of 295px has been introduced. These changes optimize the container element's height for smaller screens.

🐇💻

In the land of code, where the semicolons lie,

Changes were made, under the GitHub sky.

Height was tweaked, for screens small and neat,

Now our site looks even more sweet! 🎉🥕


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • Mention @coderabbitai in any review comment for bot assistance.
  • Note: Review comments are made on code diffs or files, not on the PR overview.

Pause Incremental Reviews

  • Insert @coderabbitai: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.
  • Additionally, enter @coderabbitai pause to halt the bot's ongoing reviews as any issue or review comment. To
    resume reviews, enter @coderabbitai resume as any issue or review comment.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 0

Commits Files that changed from the base of the PR and between 8c79253 and b2208bd.
Files selected for processing (1)
  • src/components/HomepageFeatures/styles.module.css (1 hunks)
Files skipped from review due to trivial changes (1)
  • src/components/HomepageFeatures/styles.module.css

@himughanakash
Copy link
Contributor Author

please add hacktoberfest tag when merge this please

@FrancescoXX FrancescoXX merged commit 6260b4b into FrancescoXX:main Oct 15, 2023
1 check passed
Copy link

github-actions bot commented Mar 9, 2024

🎉 This PR is included in version 0.14.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI Issue]: Box size height is too long according to the content in boxes
2 participants