-
Notifications
You must be signed in to change notification settings - Fork 66
[80] Fixes for Hero component #97
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
Conversation
Since I haven't done this yet. @all-contributors please add @rmbarley for Code |
I've put up a pull request to add @rmbarley! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution, and congrats on your PR being approved!
We’re excited to have your work as part of Open Code Chicago’s Hacktoberfest 2025 project.
Pull Request
Thanks for contributing to Hacktoberfest 2025!
Description
Hero must start after the header (add top margin / offset). It must not sit under the header. (OBE)
CTA button is not visible — implement it to match the design (size, color, text, placement). (OBE)
index.css
Slider controls/buttons don’t match the design — update styling and spacing.
Slider behavior:
Active pagination button should be outlined (only outline for current slide).~
Optional advanced: add a loading/progress state on the active pagination showing progress (e.g., 100% over 8s).
Checklist