feat: create Hero component for the landing page, including navigatio… - #129
feat: create Hero component for the landing page, including navigatio…#129broskell wants to merge 3 commits into
Conversation
…n, dynamic data fetching, and external links.
|
@broskell is attempting to deploy a commit to the Sahil's projects Team on Vercel. A member of the Team first needs to authorize it. |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request focuses on a UI/UX enhancement for the landing page. It transforms the previously plain text "Contributors" link into a visually distinct and interactive button, aiming to improve its discoverability and encourage engagement with the open-source community. The changes ensure a more modern aesthetic and better user experience while maintaining compatibility with the existing Tailwind styling system. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request improves the styling of the 'Contributors' link by turning it into a button. The implementation uses a long string of Tailwind utility classes directly in the component. While this works, for better maintainability and reusability, I've suggested extracting these styles into a dedicated CSS class using @apply, as was also mentioned in the pull request description.
…ggle, and dynamic memory count, along with initial global styling.
…ew ContributorsButton.
|
Successfully created a global, reusable ContributorsButton component (src/components/ContributorsButton.jsx) and updated Hero.jsx to use it. The button maintains its desktop text and mobile icon styles, matching your UI design while being exportable to other components. |
broskell
left a comment
There was a problem hiding this comment.
Successfully created a global, reusable ContributorsButton component (src/components/ContributorsButton.jsx) and updated Hero.jsx to use it. The button maintains its desktop text and mobile icon styles, matching your UI design while being exportable to other components.
Pull Request Description
Summary
This pull request improves the visual styling of the Contributors button in the landing page navbar.
A new reusable
.btn-contributorsCSS class was added and applied to the Contributors link inHero.jsxto make it more modern, visually prominent, and interactive.Type of Change
Related Issues
Motivation and Context
Previously, the Contributors link appeared as plain text in the navigation bar. Since the Contributors page is important for open-source participation, making it visually distinct improves discoverability and enhances the overall landing page design.
Changes Made
.btn-contributorsstyle insrc/index.csssrc/pages/hero/Hero.jsxScreenshots (if applicable)
Before:
Plain text "Contributors" link in navbar.
After:
Styled Contributors button with improved visibility and hover interaction.
Testing
Code Quality
Deployment
Breaking Changes
This change does not introduce any breaking changes.
Additional Notes
The styling changes were implemented using Tailwind-compatible CSS and do not modify any existing functionality.
Screenshots
Before

*After
