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

Design imp #1079

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ItsHamidMustafa
Copy link

Description

I enhanced the UI of the Hero section and cards and added a ScrollToTopButton component. Below is a breakdown of the changes:

Centered the Hero Section: The Hero section is now centered for a more balanced and aesthetically pleasing layout.
Added Rounded Borders: Rounded the borders of both the Hero section and the cards to create a softer visual effect.
Glow Effect: Added a subtle glow effect to the cards to make them stand out and add a modern touch.
Scroll-to-Top Button: Created a ScrollToTopButton component (in ScrollToTopButton.jsx) that smoothly scrolls to the top of the page. This component has not yet been added to the main layout file.

Notes

Since this was my first time working with TypeScript and Next.js, I followed the project’s existing structure, although I’m used to React projects where the main component file is typically App.jsx.
I can integrate the ScrollToTopButton into the main layout if needed, or leave it as a standalone component for others to add.

Let me know if any further adjustments or explanations are needed!

Rounded Hero Design, because it looks modern and neat.
Rounded Cards look more neat
Added a scroll to top button, haven't styled it because I don't know how to add it to the main file, typically named App.jsx in react projects. This is my first time working in TypeScript and Next.js
@ItsHamidMustafa ItsHamidMustafa requested a review from a team as a code owner November 15, 2024 03:28
@CLAassistant
Copy link

CLAassistant commented Nov 15, 2024

CLA assistant check
All committers have signed the CLA.

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.

2 participants